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.

Sign up for GitHub

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

Jump to bottom

Move test fixtures to lib #16

Merged
merged 46 commits into from
Jun 4, 2020
Merged

Move test fixtures to lib #16

merged 46 commits into from
Jun 4, 2020

Conversation

charlieparkes
Copy link
Collaborator

Drop the lpipe.testing module in favor of the boto3-fixtures library

charlieparkes and others added 30 commits May 15, 2020 15:38
@charlieparkes
Add project_urls to setup()
ace5eee
@charlieparkes
Update README.md
f68b629
@charlieparkes
Update README.md
ddd4f43
@charlieparkes
Run unit and integration tests at the same time to fix codecov
077e9ff
@charlieparkes
Split return value handling into separate function return_handler()
8101a64
@charlieparkes
Fix parameters required by return_handler to split that code apart
23581f5
@charlieparkes
Split pipeline into smaller functions. Move interfaces with thirdpart…
c9c08a1 
…ies to contrib/. Move dict tools to contrib/mindictive
@charlieparkes
Make utils.repr() render a string which is more likely to be usable t…
009d9da 
…o create the same object again
@charlieparkes
Document include_all_params argument for Action class
23a8f40
@charlieparkes
Fix import of utils.repr in lpipe.logging
94c9499
@charlieparkes
Add type hints everywhere
70c17ff
@charlieparkes
Fix tabs -> spaces
3431204
@charlieparkes
Merge pull request #4 from mintel/reduce_complexity
9b4f5a0 
Reduce complexity, add docs, move integrations to contrib
@charlieparkes
Move boto3 proxy to contrib (#6)
582fd9f 
* Move _boto3 module to contrib.boto3

* Fix import for emit_logs in testing.awslambda

* Change travisci tests command to just 'make test'

* Fix imports to various utils in testing.awslambda
@charlieparkes
Remove default import and usage of sentry and allow user to pass any …
5a8c8a4 
…exception handler function in it's place
@charlieparkes
Don't require sentry in lpipe or test it in main tests
7453b6b
@charlieparkes
Add sentry-sdk to dev dependencies
91c96ca
@awichmann-mintel
fix sqs.delete_message_batch (#8)
a7278cd
@charlieparkes
Merge pull request #7 from mintel/use_any_exception_handler
0617ec3 
Use any exception handler
@charlieparkes
Don't require sentry in lpipe or test it in main tests
66503e1
@charlieparkes
Disable integration tests since they aren't checking anything useful …
2b7f423 
…to lpipe. Testing the test fixtures themselves will move to a different library.
@charlieparkes
Improve test coverage of contrib.boto3
783c7d7
@charlieparkes
Improve coverage of pipeline and signature modules
f4d57d4
@charlieparkes
Move major classes to their own modules, import all major classes at …
2a3addb 
…top level module
@charlieparkes
fmt
0e1b73d
@charlieparkes
Update lpipe.__init__ docstring to match github repo description
4fb75b2
@charlieparkes
fmt
b514bff
@charlieparkes
lock env
7e9e0c7
@charlieparkes
Fix linting errors
ca2cb6b
@charlieparkes
Merge pull request #9 from mintel/raise_coverage
eacf316 
Raise coverage
@charlieparkes
Remove contrib.taxonomy, as it's specific to Mintel (#10)
a918802
@charlieparkes
Remove lpipe.testing.sqs._sqs_queue_exists() since it's legacy code a…
2176568 
…nd is completely unused
@charlieparkes
Add setup(extras_require) for sentry exception_handler (#11)
27486b0
@charlieparkes
Add tests/unit/test_normalize.py
67328c7
@charlieparkes
Test generate enum, autoencoder, all normalize functions, and refacto…
babce80 
…r to simplify process_event test fixtures
@charlieparkes
Add dummy exception_handler to process_event fixture-ized tests, and …
a4ce455 
…remove records_not_a_list fixture because the way tests auto-encode will always be a list
@charlieparkes
fmt
dbd7465
@charlieparkes
Fix all flake8 errors
f33242e
@charlieparkes
Remove pytest-localstack, travisCI docker service
adad75b
@charlieparkes
Rename uses of {service}_moto with {service}
4c210dc
@charlieparkes
Merge pull request #13 from mintel/remove_docker_and_localstack
d024a33 
Remove docker and localstack
@charlieparkes
Remove Jenkinsfile
52ec0c2
@charlieparkes
Add tests for mindictive.get_nested on objects
0a9bf23
@charlieparkes
Update manifest to reflect what's actually in the repo
e453f7b
@charlieparkes
Update package __summary__
6ba464c
@charlieparkes
Remove lpipe.testing, add boto3_fixtures and use that instead
06e7fe8
@charlieparkes charlieparkes merged commit c14d268 into master Jun 4, 2020
@charlieparkes charlieparkes deleted the move_test_fixtures_to_lib branch June 4, 2020 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@charlieparkes @awichmann-mintel