Skip to content

test: add tests for Smiles class - #13

Merged
thomasrockhu-codecov merged 1 commit into
mainfrom
th/smiles-tests
Oct 4, 2026
Merged

thomasrockhu-codecov merged 1 commit into
mainfrom
th/smiles-tests

Conversation

@thomasrockhu-codecov

Copy link
Copy Markdown
Contributor

Summary

  • Add app/test_smiles.py covering Smiles.smile().

Stacked on #12 (th/smiles), which introduces the Smiles class.

Test plan

  • python -m pytest --cov=app — 5 passed, app/smiles.py at 100% coverage

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@thomasrockhu-codecov

Copy link
Copy Markdown
Contributor Author

Harness Code Coverage report

✅ Patch coverage is 100.00%.
✅ Project coverage is 100.00%. Comparing base (dba612a) to head (be67693).

Additional details and impacted files
@@          Coverage       Diff          @@
##         th/smiles        #13      +/- ##
===========================================
+ Coverage    78.57%    100.00%  +21.43%   
===========================================
  Files            2          3       +1   
  Lines           14         17       +3   
===========================================
+ Hits            11         17       +6   
- Misses           3          0       -3   
  Partials         0          0            

@thomasrockhu-codecov
thomasrockhu-codecov changed the base branch from th/smiles to main October 4, 2026 07:50
@thomasrockhu-codecov
thomasrockhu-codecov merged commit 17ff95a into main Oct 4, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant