asjoyner / shade

SHA Drive Engine stores files in the cloud, in a flexible fashion, optionally encrypted.
Apache License 2.0
7 stars 3 forks source link

More cleanup and some reorganization #4

Closed shanel closed 8 years ago

shanel commented 8 years ago

Did some more meta-linting. In addition I also tried to remove "stuttering" of package and type naming. Specifically this was needed in the drive/ directory. One thing that I am having some trouble figuring out is why the tests fail for the fusefs stuff. They appear to not be able to find definitions of two methods which are most definitely defined:

./fuse_test.go:78: file.ToJSON undefined (type shade.File has no field or method ToJSON) ./fuse_test.go:110: undefined: cache.NewReader

Not sure if this is something I did or if it is in the original...

asjoyner commented 8 years ago

The tests fail on Travis CI with the following error: config_test.go:126: test "single localdrive provider" wanted error "" but got "unsupported provider in config: \"localdrive\""

I don't see a failure there for ToJSON or NewReader? Maybe a merge fail, or need to pull from head?

https://travis-ci.org/asjoyner/shade/builds/145046275 https://travis-ci.org/asjoyner/shade/jobs/145046277

asjoyner commented 8 years ago

It's clear why the Travis CI localdrive test is failing. I'll fix that, and give names to the anonymous structs post merge. Thanks for more fixes, Shane!