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

Add integration tests #66

Open
wants to merge 52 commits into
base: add-tests
Choose a base branch
from
Open

Add integration tests #66

wants to merge 52 commits into from

Conversation

frostming
Copy link
Member

No description provided.

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
- Also clean up python specifiers

Signed-off-by: Dan Ryan <[email protected]>
- Also add magic methods to MetaSet object
- Super fast and efficient now due to set comparisons
- Old method had sometimes thousands of the same pyspecs
- This consolidates as it builds

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>

Test fixes

Signed-off-by: Dan Ryan <[email protected]>

Fix specifier dedup logic

Signed-off-by: Dan Ryan <[email protected]>

Fix import

Signed-off-by: Dan Ryan <[email protected]>

Fix tox

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>

Add set operations and comparison methods to metasets

Signed-off-by: Dan Ryan <[email protected]>

Add abstract extraction methods for marker cleanup

Signed-off-by: Dan Ryan <[email protected]>

Add intersection method for PySpecs and smarter unions and creation

Signed-off-by: Dan Ryan <[email protected]>

Simplify and cleanup metadata implementation

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
@codecov
Copy link

codecov bot commented Sep 20, 2019

Codecov Report

❗ No coverage uploaded for pull request base (add-tests@3ed2038). Click here to learn what that means.
The diff coverage is 48.89%.

Impacted file tree graph

@@             Coverage Diff              @@
##             add-tests      #66   +/-   ##
============================================
  Coverage             ?   54.72%           
============================================
  Files                ?       42           
  Lines                ?     2701           
  Branches             ?      475           
============================================
  Hits                 ?     1478           
  Misses               ?     1108           
  Partials             ?      115
Impacted Files Coverage Δ
src/passa/cli/install.py 0% <ø> (ø)
src/passa/cli/options.py 78.43% <ø> (ø)
src/passa/cli/add.py 0% <ø> (ø)
src/passa/internals/_pip_shims.py 72.97% <ø> (ø)
src/passa/cli/remove.py 0% <ø> (ø)
src/passa/actions/sync.py 0% <ø> (ø)
src/passa/cli/upgrade.py 0% <ø> (ø)
src/passa/models/environments.py 22.67% <0%> (ø)
src/passa/models/providers.py 61.29% <0%> (ø)
src/passa/internals/utils.py 64.91% <0%> (ø)
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ed2038...4097493. Read the comment docs.

@frostming
Copy link
Member Author

frostming commented Sep 26, 2019

Creating virtualenv for every test case is a big performance killer in testing.
I am considering to design a new (un)installation interface so that we can get a full mocked (un)installation and get rid of venv.

@frostming
Copy link
Member Author

Whoa, after mocking installation, the time cost is reduced by 60%!

@frostming frostming marked this pull request as ready for review September 28, 2019 13:25
@frostming
Copy link
Member Author

frostming commented Sep 28, 2019

I am not apt to make huge PRs, so if the review is done we can start merging stuff. No need to have a full test suite at this point. Especially there are some bugfixes that are worth merging into the master. Do we need to cherry-pick?

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.

2 participants