Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cloudbees Folder Support #108

Closed
terabyte opened this issue Oct 22, 2015 · 6 comments
Closed

Cloudbees Folder Support #108

terabyte opened this issue Oct 22, 2015 · 6 comments
Milestone

Comments

@terabyte
Copy link
Contributor

Hey guys,

Just a heads-up, I am working on folder support. Here is a snapshot of what I have so far:
213bb62

If you have any advice/feedback about how you'd like it done, feel free to chime in, but don't feel like you need to waste any time on it necessarily until I'm done. I've examined the code pretty carefully and I think I am making decisions similar to the structure of other things fairly consistently, so hopefully it will be an easy PR to take.

My basic approach so far is adding a FolderJob object, and JenkinsServer.getFolder(Job someJob) which returns a FolderJob object (regardless of job type), on which you can call isFolder() to see if it is a valid folder. Then, on the FolderJob, you can call getJobs() to get the jobs under it.

I'll also add JenkinsServer.createFolder() and FolderJob.createFolder(), which will probably throw an error if you don't have the cloudbees plugin installed.

The linked history also includes some bugfixes and refactorings.

@terabyte
Copy link
Contributor Author

So, I have a nearly-completed change, but I seem to have broke the integration tests, so it's gonna be Monday probably.

In related news: nice tests ya got there. Many thanks.

@khmarbaise
Copy link
Member

Take your time...and write good tests ;-)

@terabyte
Copy link
Contributor Author

terabyte commented Nov 2, 2015

Here is the newest version, I suspect I'm very close to being ready to PR:
master...terabyte:feature/folder-support

I tried to write an integration test, but I can't figure out how to install the cloudbees plugin in the embedded jenkins instance. Any help there? Basically, the "UpdateManager" for that instance can't see any plugins. I could download a version of the cloudbees HPI plugin file and install it that way, or I could add the live update site and have it download, but I can't figure out how to programmatically do either of those...

@khmarbaise
Copy link
Member

The problem with the integration tests is a known to me what i don't like...I haven't found a way either yet...

@terabyte
Copy link
Contributor Author

blah...that's bad news =( For the purposes of this change, should we accept it as-is and make a "todo" to try and write better integration tests later? Or is there something else I can do shorter-term to make this change more palatable?

@khmarbaise khmarbaise added this to the Release 0.3.3 milestone Nov 27, 2015
@khmarbaise
Copy link
Member

Merged into 0.3.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants