Dashboard SQLException and Travis system test fail

[keshavsethi]

An error was coming in the dashboard and system tests were also failing.

I work fine after correcting the typo

Thanks!

[gitpod-io]

[codecov]

Codecov Report

Merging #8186 into main will decrease coverage by 1.81%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #8186      +/-   ##
==========================================
- Coverage   81.69%   79.88%   -1.82%     
==========================================
  Files         100      100              
  Lines        5846     5866      +20     
==========================================
- Hits         4776     4686      -90     
- Misses       1070     1180     +110     

Impacted Files	Coverage Δ
app/models/tag.rb	97.50% <100.00%> (ø)
app/channels/application_cable/channel.rb	0.00% <0.00%> (-100.00%)	⬇️
app/channels/application_cable/connection.rb	0.00% <0.00%> (-100.00%)	⬇️
app/channels/user_channel.rb	0.00% <0.00%> (-83.34%)	⬇️
app/channels/user_notification_channel.rb	0.00% <0.00%> (-83.34%)	⬇️
app/channels/room_channel.rb	0.00% <0.00%> (-71.43%)	⬇️
app/controllers/subscription_controller.rb	59.59% <0.00%> (-11.12%)	⬇️
app/controllers/questions_controller.rb	81.57% <0.00%> (-9.22%)	⬇️
app/models/tag_selection.rb	88.00% <0.00%> (-8.00%)	⬇️
app/controllers/tag_controller.rb	76.14% <0.00%> (-5.20%)	⬇️
... and 9 more

[cesswairimu]

Just saw the system failures on travis on the recent PRs... Great job catching this @keshavsethi and opening this so quick 🎉 🎉 Thanks

[keshavsethi]

Thanks!! @cesswairimu it is because of some chrome driver version.

Minitest::UnexpectedError:         Selenium::WebDriver::Error::SessionNotCreatedError: session not created: Chrome version must be between 70 and 73
          (Driver info: chromedriver=73.0.3683.68 (47787ec04b6e38e22703e856e101e840b65afe72),platform=Linux 4.15.0-1077-gcp x86_64)

[cesswairimu]

Which is strange we haven't changed the version since February 🤔 not sure why we are getting the failures now...do we need to try 73.0.0?

[keshavsethi]

@cesswairimu @jywarren @alaxalves @icarito System Tests are failing because of the chrome driver version issue. Please check!!
Thanks!!

[cesswairimu]

From the latest travis-ci update notes it shows that Google Chrome changed to 83.0.4103.116-1..and the driver for this should be used with ChromeDriver 83.0.4103.39 or ChromeDriver 83.0.4103.14 but none of these seem to work 🤔 Not sure where we can get the exact chrome version that we are using

[keshavsethi]

@cesswairimu @jywarren I tried a few things but still, errors are coming 🤔 Please check!!
Thanks!!

[cesswairimu]

I am going to try a brute-force from version 73 upwards see if anything works...opening on a new PR #8197

[jywarren]

Oh gosh, this is pretty bad. Thanks for looking into this! I will check it out too to see if we can figure it out.

In the meantime, LOTS of info and guidance on Chromedriver here: #6095

[jywarren]

This note refers to work by @alaxalves on this which helped us mitigate the chrome version moving target... perhaps related?

So the latest is that @alaxalves was able to pin to a 3rd party repository of Chrome versions, at https://github.com/webnicer/chrome-downloads, in #6099. It has Chrome 78, latest updated version 4 days ago, so this seems viable...

So we're pinned at Chrome 75 ChromeDriver 74:

[jywarren]

Was this resolved? I see @cesswairimu closed this attempt: #8197

[jywarren]

Ah i see finally in #8203 - thanks!