Skip to content

Tests for the github bridge - #706

Merged
rng-dynamics merged 6 commits into
masterfrom
dev-github-bridge-tests
Sep 14, 2021
Merged

rng-dynamics merged 6 commits into
masterfrom
dev-github-bridge-tests

Conversation

@rng-dynamics

Copy link
Copy Markdown
Contributor

I wrote an integration test for the github bridge. It creates a few issues, comments, labels, edits a comment, and uses two identities. It is a simple "happy path" of an import. If you like it, we can extend it to cover more code of the importer.

This request depends on #694, i.e., #694 should be merged first.

@rng-dynamics
rng-dynamics marked this pull request as draft September 5, 2021 15:02
@rng-dynamics

Copy link
Copy Markdown
Contributor Author

I will mark this as a draft as long as #694 is not merged.

@rng-dynamics
rng-dynamics force-pushed the dev-github-bridge-tests branch from 43cc33f to 8d38e3d Compare September 5, 2021 15:15
@MichaelMure

Copy link
Copy Markdown
Contributor

@rng-dynamics go ahead and merge what you feel makes sense for the github bridge. You know better than anyone what makes sense there :)

@MichaelMure

Copy link
Copy Markdown
Contributor

Again, sorry but I just can't be a proper project lead currently. I'll come back to it though.

@rng-dynamics
rng-dynamics marked this pull request as ready for review September 13, 2021 19:01
@rng-dynamics

Copy link
Copy Markdown
Contributor Author

@MichaelMure, I would need someones approval to merge. (Same as in #694.)

@MichaelMure

Copy link
Copy Markdown
Contributor

Ha right, that should be good now.

@6543

6543 commented Sep 13, 2021

Copy link
Copy Markdown
Contributor

@MichaelMure I can help you to make this more (helthy) communety driven - but I can't step fully in ... since I already do handle a lof of other projects :)

@6543

6543 commented Sep 13, 2021

Copy link
Copy Markdown
Contributor

@MichaelMure PS: you can PM me via mail or matrix (@marddl:obermui.de) ...

@6543 6543 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

manually tested -> worked

@rng-dynamics
rng-dynamics merged commit d546cde into master Sep 14, 2021
@rng-dynamics
rng-dynamics deleted the dev-github-bridge-tests branch September 14, 2021 20:24
@rng-dynamics

Copy link
Copy Markdown
Contributor Author

thanks guys

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