Remove excessive namespacing in tests
Fix indentation
Fix a migration, which probably would have destroyed manual approvals
Pass around License objects instead of names
Always pass around License objects, never names
Move name matching logic back to License

Based on a simpler version of building whitelisted licenses.
Make private readers for some ivars
Make private method private
Reorder methods
Correct test description
Merge branch 'master' into pass-license-objects-v2

Conflicts:
	lib/license_finder.rb
	lib/license_finder/dependency_manager.rb
	lib/license_finder/license.rb
	lib/license_finder/license/definitions.rb
	lib/license_finder/tables/dependency.rb
	lib/license_finder/yml_to_sql.rb
	spec/lib/license_finder/dependency_manager_spec.rb
	spec/lib/license_finder/reports/html_report_spec.rb
	spec/lib/license_finder/reports/markdown_report_spec.rb
	spec/lib/license_finder/tables/dependency_spec.rb
	spec/lib/license_finder/yml_to_sql_spec.rb
Make class method name match instance method names
Remove dead code
Don't blow up in a somewhat unusual case

Was blowing up when the system had a whitelist and there was a
Dependency without a name (probably it had just been created).
Fix test broken by using canonical license names
Merge pull request #84 from mainej/pass-license-objects

Pass License objects, not names