use `locale` as the path in partner repacks.

mozilla-releng / beetmoverscript

An implementation of scriptworker for moving TC artifacts to all release-related buckets.

1 stars 19 forks source link

use `locale` as the path in partner repacks. #119

Closed escapewindow closed 6 years ago

escapewindow commented 6 years ago

This should hopefully get our beetmover partner repacks working.

Originally, we thought we would write the artifacts to a path, and preserve that relative path when beetmoving. However, public and private paths differ.

Instead, I'm overloading the locale attribute for the path to use. We should probably do this in a different way, but it looks like it's working so far.

Creating the PR for coverage testing, since the get_hash test seems to break on me locally on osx.

escapewindow commented 6 years ago

~i want to add more coverage~
~I'm a little worried about not sanity checking the paths at all, and only relying on the tree to do the right thing.~

We need some more branch coverage, but other than that we're good.