moved test class to correct package and deleted Locksmith. As a result of the delete, caching happens directly in lock class, which makes sense because i want each lock instantiation to return the same instance for the same id. Also, the lock doesn't need to be byteable