Fix the integration tests

[DilumAluthge]

No description provided.

[DilumAluthge]

cc: @ablaom

[ablaom]

Thanks for looking further into this. Indeed the flag is now being set and the integration tests are definitely running here. However, my intention with this condition was to rule out testing except for PRs with dev as source. Since the current PR does not have dev as source, I don't want them to run here. So why is my flag nevertheless getting set?

[ablaom]

BTW. I forgot that I hard coded the tests to be minimal, for testing the tests, so they only take a few minutes not an hour.

[DilumAluthge]

Seems like there's a bug in the condition.

[ablaom]

Mmmm. I literally copied the condition from here where it appears to work.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fc139f2) 57.89% compared to head (285147e) 57.89%.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev    #1081   +/-   ##
=======================================
  Coverage   57.89%   57.89%           
=======================================
  Files           2        2           
  Lines          38       38           
=======================================
  Hits           22       22           
  Misses         16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[DilumAluthge]

@ablaom Looks like it's working now.

[ablaom]

Thanks for your time @DilumAluthge working this out. Much appreciated. I'll merge this and give it another whirl with a new dev -> master PR.