Only allow textId or scriptureRange in a buildID.

[johnml1135]

This change is 

[ddaspit]

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

---

src/Serval.Translation/Services/EngineService.cs line 238 at r1 (raw file):

                            {
                                throw new InvalidOperationException(
                                    $"The corpus {c.Id} cannot specify both 'textIds' and 'scriptureRange' for pretranslate"

You should end exception messages with a period. Also, it would be good to put quotes around "pretranslate", since that is a JSON property.

[johnml1135]

src/Serval.Translation/Services/EngineService.cs line 238 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

You should end exception messages with a period. Also, it would be good to put quotes around "pretranslate", since that is a JSON property.

Done.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 16.66667% with 10 lines in your changes missing coverage. Please review.

Project coverage is 71.21%. Comparing base (f070a15) to head (a17052a).

Files	Patch %	Lines
src/Serval.Translation/Services/EngineService.cs	16.66%	8 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #401      +/-   ##
==========================================
- Coverage   71.31%   71.21%   -0.10%     
==========================================
  Files         151      151              
  Lines        7177     7189      +12     
  Branches     1124     1126       +2     
==========================================
+ Hits         5118     5120       +2     
- Misses       1604     1612       +8     
- Partials      455      457       +2     

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

[ddaspit]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @johnml1135)