Skip to content

reorganize examples cross-imports to simplify tests #157

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

Merged
merged 1 commit into from
May 29, 2019
Merged

reorganize examples cross-imports to simplify tests #157

merged 1 commit into from
May 29, 2019

Conversation

mattsb42-aws
Copy link
Member

Description of changes:
In preparation for #156, I realized that I hadn't given these examples tests the same treatment that I gave the DDB encryption client and Pipeformer.

These changes make writing tests for examples much simpler and less finicky.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mattsb42-aws mattsb42-aws requested a review from a team May 29, 2019 19:17
@mattsb42-aws
Copy link
Member Author

It looks like the AppVeyor CPython34-x64 runner is still failing. :( Best we've been able to tell, this is a bug on their end. We need to figure that out eventually, but not necessarily now.

@lizroth lizroth merged commit 433adfd into aws:master May 29, 2019
@mattsb42-aws mattsb42-aws deleted the examplespath branch May 29, 2019 21:26
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