Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

testing #2

Merged
merged 18 commits into from
Mar 30, 2021
Merged

testing #2

merged 18 commits into from
Mar 30, 2021

Conversation

CarloDePieri
Copy link
Owner

  • change toolchain to poetry, refactor tasks.py
  • add functional test for the script in a graphical environment
  • add test framework and ~50% test coverage
  • fix backup fixture when missing config_file
  • increase unit test coverage ~87%
  • reach 100% unit test coverage for the library
  • change the exception name so that pytest does not think it's a test
  • add unit tests for the cli module
  • add coveralls dev dependency
  • add missing coveralls dependency
  • fix missing check on existing .gmail.json, add ENV credentials support
  • add missing pyperclip mock
  • add prod github workflow, add act workflow
  • add dev github/act workflow
  • fix apt/add-apt in github workflows not running as root
  • fix wrong permission with apt update, separate add-apt from apt update
  • change how python is provided in ci, fixed missing env in act-shell
  • update prod workflow, update README.md

@CarloDePieri CarloDePieri merged commit aec9e28 into master Mar 30, 2021
@0xArab
Copy link

0xArab commented May 30, 2021

Brother Can you add documentation ?

i didn't see any documentation in pymailltm

@CarloDePieri
Copy link
Owner Author

@0xArab there is no documentation yet.

Thing is, I'm not really happy with the state of the project: I would love for the api wrapper to cover 100% of the web api functionalities and for the api wrapper code to be separated from the ui code.

This will probably entail a substantial refactor (that will gradually deprecate old modules); after that I plan to write the needed documentation.

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.

None yet

2 participants