use consistent class and file names

[NBS-LLC]

Decided between the short name:

• controller.ts vs treasures-controller.ts
• if imports need to be renamed to be more specific
• ensure class and file names match
• consider how multiple APIs with short names might coexist in app.ts

[NBS-LLC]

Might be better not to use short names to better align code with unit test results:

  TreasureController
    getRandomTreasure()
       should return a treasure object

  GET /api/treasures/random
     should return random treasure data (5278ms)

  Treasure
    getAttributes()
       should return an empty array on initialization
    addAttribute()
       should add an attribute to the treasure