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

[FormatImplementationTest] added general unit tests for format implementations #1008

Merged

Conversation

fulmeek
Copy link
Contributor

@fulmeek fulmeek commented Jan 12, 2019

This adds general unit tests for the format implementations.

Note that this does not test whether the output of the formats are valid as they are subject of specific tests.

@fulmeek fulmeek force-pushed the upstream-FormatImplementationTest branch from 4dedacb to b887251 Compare January 12, 2019 19:49
@logmanoriginal
Copy link
Contributor

This works well too. Same as #1007, the variable $bridges should be changed to $formats to avoid confusion.

@fulmeek
Copy link
Contributor Author

fulmeek commented Feb 3, 2019

Thanks again, this should do it :)

@logmanoriginal logmanoriginal merged commit 11a39af into RSS-Bridge:master Feb 4, 2019
@logmanoriginal
Copy link
Contributor

Merged, thanks!

@fulmeek fulmeek deleted the upstream-FormatImplementationTest branch February 4, 2019 22:29
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
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