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

Improved [MicroBadger] tests speed #2512

Merged
merged 4 commits into from
Dec 12, 2018
Merged

Conversation

PyvesB
Copy link
Member

@PyvesB PyvesB commented Dec 11, 2018

Addresses #2489. Selected images that are less "heavy" and seemed to give faster response times from the API. I also aligned the example previewUrls to match what was used in the test.

@PyvesB PyvesB changed the title Improved tests and examples speed Improved [MicroBadger] tests speed Dec 11, 2018
@shields-ci
Copy link

shields-ci commented Dec 11, 2018

Messages
📖 ✨ Thanks for your contribution to Shields, @PyvesB!

Generated by 🚫 dangerJS against 529c195

@paulmelnikow
Copy link
Member

It's not essential, though it'd be great if you could update the previewUrls to namedParams to help with #2496.

@paulmelnikow paulmelnikow added the keep-service-tests-green Related to fixing failing tests of the services label Dec 11, 2018
paulmelnikow
paulmelnikow previously approved these changes Dec 11, 2018
@PyvesB
Copy link
Member Author

PyvesB commented Dec 11, 2018

No tests have timed out this time round and they execute much faster with the new images, below one second.

@PyvesB
Copy link
Member Author

PyvesB commented Dec 11, 2018

@paulmelnikow can these examples really be converted at this point? As the tag is optional, wouldn't we be affected by #2497?

@paulmelnikow
Copy link
Member

In examples I haven't been using optionals, so #2497 doesn't apply. Instead there's usually a different pattern for each example, one with and without the optional part. That way each example renders the way you'd expect:

screen shot 2018-12-11 at 5 08 48 pm

@PyvesB
Copy link
Member Author

PyvesB commented Dec 11, 2018

I got confused with the route pattern, it's getting quite late over here. The examples should now be migrated.

@@ -22,22 +22,38 @@ module.exports = class MicroBadger extends LegacyService {
return [
{
title: 'MicroBadger Size',
previewUrl: 'image-size/fedora/apache',
Copy link
Member

Choose a reason for hiding this comment

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

It looks like the patterns are missing eg image-size.

@PyvesB PyvesB merged commit 029a146 into badges:master Dec 12, 2018
@shields-deployment
Copy link

This pull request was merged to master branch. This change is now waiting for deployment, which will usually happen within a few days. Stay tuned by joining our #ops channel on Discord!

After deployment, changes are copied to gh-pages branch:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep-service-tests-green Related to fixing failing tests of the services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants