641843 - Fix folder structure for the API rewrite

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We have to update our folders to match the new structure of the mozmill-tests repository:

http://hg.mozilla.org/qa/mozmill-tests/file/default

Lets do that in milestone 2.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

I will take it.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Patch v1

Moves all the files to the appropriate folders as discussed earlier today.

Comment 3

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

Comment on attachment 526175 [details] [diff] [review]
Patch v1

Looks fine, but two comments before you do any checkins:

First, we should probably get rid of the whimboo subdir I put in--all of that code has been addressed at this point, and it causes doc artifacts when we run jsdoc-toolkit against modules/lib. 

At the very least, we should move it out to be whimboo/lib, whimboo/tests rather than what I had, tests/whimboo and lib/whimboo. That way it won't be part of the docgen. But I'd say just nuke it; we don't need it.

Second, we can't do this as a patch, though I understand it was convenient to review it that way (sorta -- I'd actually have preferred to review this as a github branch, since adding/deleting files is harder to get an overview of in patch form). 

But the merge itself needs to use git mv and then git merge or we'll lose change history on all files moved. Please do not apply this as a patch.

With those notes in mind, r=me.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to comment #3)
> First, we should probably get rid of the whimboo subdir I put in--all of that
> code has been addressed at this point, and it causes doc artifacts when we run
> jsdoc-toolkit against modules/lib. 

At this state I would like to keep at least my files from the lib folder. So I would move those up to the root for the time being (probably next 2 weeks). Then we can get rid of those. The tests have already been deleted, because they haven't added any value.

> But the merge itself needs to use git mv and then git merge or we'll lose
> change history on all files moved. Please do not apply this as a patch.

That's what I will do.

I will get it in soon.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Landed as:
https://github.com/geoelectric/mozmill-api-refactor/commit/90f030f409d87101d342081a7a85ed47cdb6b362

As you can see even with 'git mv' the history is not available on GitHub. But that really is a github only issue. The history of the files are preserved and you can reach it by using i.e. 'git log --follow lib/api/assertions.js'

Comment 6

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

Good to know. Thanks for the warning re: github history.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

That should probably also solve the question with patch, which shouldn't make any difference.