[DAFFODIL-2305] Remove some Unused Term Methods

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5.0
• Fix Version/s: 2.6.0
• Assignee: Olabusayo Kilo
• Reporter: Olabusayo Kilo
• Created: 25/Mar/20 17:30
• Updated: 27/Mar/20 12:02
• Resolved: 26/Mar/20 14:20

Description

There are some methods in Term.scala that are no longer used, and can be removed, along with their associated tests.

possibleFirstChildElementsInInfoset
The method couldBeLastElementInModelGroup, which has an abstract definition below, is also not used and it should be deleted. (The implementations of these are in ModelGroup and ElementBase.)

Ditto for possibleFirstChildTerms
Ditto for isSequendceChild
Ditto for possibleNextSiblingTerms

hasLaterRequiredSiblings is used only by unit tests, so that AND the unit tests looking at it can go away (ensure this doesn't decrease coverage if other methods are called)
Same for hasPriorRequiredSiblings.

priorSibling is unused and can be deleted.