Part of #9686 This test will run only on PRs that have 'Translation System' as a label. They are system tests only which basically will screenshot web page corresponding to file changed. To Do: send a list of files changed to tests, create a mapping of view file and corresponding route to take screenshot of only files changed, comment screenshot in PR.

@jywarren this one is not complete yet, just wanted to know if I'm on the right track ?

Codecov Report

Merging #10076 (fa629ad) into main (f41f8e7) will decrease coverage by 0.14%. The diff coverage is 85.29%.

:exclamation: Current head fa629ad differs from pull request most recent head 4744260. Consider uploading reports for the commit 4744260 to get more accurate results

@@            Coverage Diff             @@
##             main   #10076      +/-   ##
==========================================
- Coverage   82.13%   81.98%   -0.15%     
==========================================
  Files          98       98              
  Lines        5968     5974       +6     
==========================================
- Hits         4902     4898       -4     
- Misses       1066     1076      +10     

Impacted Files	Coverage Δ
app/models/tag.rb	93.26% <44.44%> (-4.27%)	:arrow_down:
app/controllers/home_controller.rb	96.05% <100.00%> (-2.62%)	:arrow_down:
app/controllers/images_controller.rb	70.27% <100.00%> (ø)
app/controllers/notes_controller.rb	85.09% <100.00%> (+0.22%)	:arrow_up:
app/controllers/tag_controller.rb	80.42% <100.00%> (ø)
app/helpers/application_helper.rb	87.62% <100.00%> (+2.21%)	:arrow_up:
app/models/image.rb	82.05% <100.00%> (-0.45%)	:arrow_down:
app/models/user.rb	86.12% <100.00%> (ø)
app/services/search_service.rb	94.89% <100.00%> (-0.21%)	:arrow_down:
... and 3 more

Code Climate has analyzed commit 4744260e and detected 0 issues on this pull request.

View more on Code Climate.

@jywarren @noi5e is there some way to restart these tests, it seems they are stuck

Noting we could trigger this by using the label: https://github.com/publiclab/plots2/labels/needs-screenshot

Really sorry this PR is pending for a long time. I'm not sure why the tests are not running. I'm not familiar with GitHub workflow so was checking how things go after adding small changes. I think opening a new PR might rerun the tests.

Yeah, this is strange that the tests aren't running. @jywarren

@imajit, something you could do is git rebase so that this PR is built off of the most current main branch. There might be some CI updates in the most current main that could help this one out.

If you've never done a rebase, I just wrote up some instructions for that this week.

I will try to close and reopen one more time

Hi :smile:, this issue has been automatically marked as stale because it has not had recent activity. Don't worry you can continue to work on this and ask @publiclab/reviewers to add in-progress label :tada: . Otherwise, it will be closed if no further activity occurs in 10 days -- but you can always re-open it if you like! :100: Thank you for your contributions! :raised_hands: :balloon:.