Automatically en-/dis-able definitions

[imnasnainaec]

Resolves #2145

Once deployed, something like the following should be run on the server:

mongosh --quiet CombineDatabase --eval "db.ProjectsCollection.updateMany({}, {\$set: {'definitionsEnabled': false}})" && mongosh --quiet CombineDatabase --eval "db.FrontierCollection.aggregate([{\$unwind:{path:'\$senses'}},{\$unwind:{path:'\$senses.Definitions'}},{\$group:{_id:'\$projectId'}}])" | grep -Eoh '[0-9a-z]{24}' | while read p || [[ -n $p ]]; do mongosh --quiet CombineDatabase --eval "db.ProjectsCollection.updateOne({_id: {\$eq: ObjectId('$p')}}, {\$set: {'definitionsEnabled': true}})"; done

---

This change is 

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and project coverage change: +22.54 🎉

Comparison is base (42cbb26) 49.50% compared to head (5d3d324) 72.04%.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #2161       +/-   ##
===========================================
+ Coverage   49.50%   72.04%   +22.54%     
===========================================
  Files         294       43      -251     
  Lines        9494     3957     -5537     
  Branches      694        0      -694     
===========================================
- Hits         4700     2851     -1849     
+ Misses       4239     1106     -3133     
+ Partials      555        0      -555     

Flag	Coverage Δ
backend	72.04% <100.00%> (+0.04%)	⬆️
frontend	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
Backend/Controllers/LiftController.cs	57.66% <100.00%> (+0.79%)	⬆️
Backend/Services/LiftService.cs	93.85% <100.00%> (+0.03%)	⬆️

... and 250 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jmgrady]

Reviewed 3 of 11 files at r1, 10 of 10 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @imnasnainaec)

---

Backend.Tests/Controllers/LiftControllerTests.cs line 489 at r2 (raw file):

            Assert.That(proj.DefinitionsEnabled, Is.True);
        }

The only difference that I can see between these two functions is the file to be imported and the expected result. Wouldn't it be easier to maintain if there was a single function to perform this task which receives the filename and expected result as arguments?

[jmgrady]

Reviewed 1 of 1 files at r3, 1 of 1 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @imnasnainaec)

[imnasnainaec]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @jmgrady)

---

Backend.Tests/Controllers/LiftControllerTests.cs line 489 at r2 (raw file):

Previously, jmgrady (Jim Grady) wrote…

The only difference that I can see between these two functions is the file to be imported and the expected result. Wouldn't it be easier to maintain if there was a single function to perform this task which receives the filename and expected result as arguments?

Done.

[jmgrady]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)