Updated get_books

[TaperChipmunk32]

get_books now accepts ";" as a delimeter and allows for book ranges, just as get_chapters does.

---

This change is 

[TaperChipmunk32]

@ddaspit @johnml1135 The tests are failing because I chose to raise a ValueError on an invalid id, instead of a RuntimeError, to be consistent with get_chapters and parse_selection. Should I just update the exceptions to be RuntimeErrors or should the tests be updated? I assume the former, but I feel that ValueError is a more informative and consistent error message.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.14%. Comparing base (6d695f9) to head (5f29642).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   88.12%   88.14%   +0.01%     
==========================================
  Files         273      273              
  Lines       15989    16010      +21     
==========================================
+ Hits        14091    14112      +21     
  Misses       1898     1898              

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

[ddaspit]

Reviewed 1 of 1 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @TaperChipmunk32)

---

machine/scripture/parse.py line 45 at r2 (raw file):

def update_selection(book_set: Set[int], book_nums: Set[int], subtraction: bool) -> Set[int]:

This should be prefixed with an underscore, since it is a "private" function.

---

machine/scripture/parse.py line 50 at r2 (raw file):

            book_set.difference_update(book_nums)
        else:
            book_ids = set()

This can be simplified using list comprehension.

[ddaspit]

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