Skip to content

Run Tahoe tests on TravisCI (part 2) - #479

Merged
OmarIthawi merged 1 commit into
appsembler/tahoe/developfrom
omar/run-tahoe-tests
Nov 23, 2019
Merged

Run Tahoe tests on TravisCI (part 2)#479
OmarIthawi merged 1 commit into
appsembler/tahoe/developfrom
omar/run-tahoe-tests

Conversation

@OmarIthawi

@OmarIthawi OmarIthawi commented Nov 12, 2019

Copy link
Copy Markdown

I'm adding tests to TravisCI one by one until we have most of the tests that we'd like to have running following up to 811914e (part 1).

Ideally you could just run tox on your own device (no Docker) and that's it, but for now there's some complications which I'll explain later.

⚠️ I'm now using psycopg2-binary==2.8.3 instead of psycopg2==2.8.3 so it can work on my device without too much of issues. We've been using this library in AMC for a while and we like it.

Jira Issue: RED-434

TODO

  • Get @melvinsoft opinion on the move to psycopg-binary

@OmarIthawi

This comment has been minimized.

@OmarIthawi OmarIthawi closed this Nov 13, 2019
@OmarIthawi
OmarIthawi deleted the omar/run-tahoe-tests branch November 21, 2019 07:36
@OmarIthawi OmarIthawi changed the title (WIP) Run Tahoe tests Run Tahoe tests Nov 21, 2019
@OmarIthawi
OmarIthawi restored the omar/run-tahoe-tests branch November 21, 2019 13:10
@OmarIthawi OmarIthawi reopened this Nov 21, 2019
@OmarIthawi OmarIthawi changed the title Run Tahoe tests (WIP) Run Tahoe tests Nov 21, 2019
@OmarIthawi
OmarIthawi force-pushed the omar/run-tahoe-tests branch 2 times, most recently from 41928db to 81fde06 Compare November 22, 2019 05:36
@OmarIthawi OmarIthawi changed the title (WIP) Run Tahoe tests Run Tahoe tests on TravisCI (part 2) Nov 22, 2019
@OmarIthawi OmarIthawi changed the title Run Tahoe tests on TravisCI (part 2) (WIP) Run Tahoe tests on TravisCI (part 2) Nov 22, 2019
@OmarIthawi OmarIthawi changed the title (WIP) Run Tahoe tests on TravisCI (part 2) Run Tahoe tests on TravisCI (part 2) Nov 22, 2019
@OmarIthawi
OmarIthawi force-pushed the omar/run-tahoe-tests branch 3 times, most recently from 4d4b03c to ac40926 Compare November 22, 2019 11:47
Comment thread requirements/edx/appsembler.txt Outdated

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@melvinsoft what do you think of this switch for edx-platform? We've been using this for AMC for a while, but perhaps you have another opinion.

BTW, without this library it's very painful to run tox outside Docker.

@OmarIthawi
OmarIthawi merged commit a98304e into appsembler/tahoe/develop Nov 23, 2019
@OmarIthawi
OmarIthawi deleted the omar/run-tahoe-tests branch November 24, 2019 04:11
@OmarIthawi OmarIthawi mentioned this pull request Dec 2, 2019
6 tasks
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.

3 participants