1135070 - Testopia must set PUBLIC_METHODS for its WebServices to work

Status: RESOLVED FIXED

(Testopia :: API, defect)

Description

[u430950]

A commit introduced by bug 1090275 has resulted in all Testopia API calls not working. I'm currently working on a patch.

Comment 1

[u430950]

Attachment: patch.diff

PUBLIC_METHODS have been added to every Testopia lib/WebService file

Comment 2

[Frédéric Buclin]

With this patch, we have enough critical stuff to release Testopia 2.6 soon.

Comment 3

[u430950]

(In reply to Frédéric Buclin from comment #2)
> With this patch, we have enough critical stuff to release Testopia 2.6 soon.

I agree. With this fix's completion, Testopia retains compatibility with Bugzilla 5.0

Comment 4

[u430950]

Comment on attachment 8567143 [details] [diff] [review]
patch.diff

Changing reviewer, as LpSolit has handled Testopia updates for me in the past.

Once this has been pushed, we should package all the changes up to this point as a new Testopia release.

Comment 5

[Frédéric Buclin]

(In reply to Ryan Wilson [:f1sh] from comment #4)
> Once this has been pushed, we should package all the changes up to this
> point as a new Testopia release.

ghendricks told me that he no longer has access to the Testopia repo since they moved to git. :)

Comment 6

[u430950]

(In reply to Frédéric Buclin from comment #5)
> (In reply to Ryan Wilson [:f1sh] from comment #4)
> > Once this has been pushed, we should package all the changes up to this
> > point as a new Testopia release.
> 
> ghendricks told me that he no longer has access to the Testopia repo since
> they moved to git. :)

What is the best course of action moving forward? Do we work on getting ghendricks access to the repo, or does someone else take over?

Comment 7

[Frédéric Buclin]

No idea who manages git accounts.

Comment 8

[Frédéric Buclin]

I cannot really test this patch, because I get a lot of errors already, using Bugzilla 4.4.8. Ryan, could you test/confirm something for me?

Click the "Current Runs" in the page footer, then double-click on a run, then click on the select box for the "Build" field. I get this error:

There was an error loading the data: {"message":"Missing required parameter product_id","error":"testopia-missing-parameter","success":false}

I get the same error with the "Environment" select box.

I also get the same error under "My Cases" for the "Category" field.

Comment 9

[u430950]

(In reply to Frédéric Buclin from comment #8)
> I cannot really test this patch, because I get a lot of errors already,
> using Bugzilla 4.4.8. Ryan, could you test/confirm something for me?
> 
> Click the "Current Runs" in the page footer, then double-click on a run,
> then click on the select box for the "Build" field. I get this error:
> 
> There was an error loading the data: {"message":"Missing required parameter
> product_id","error":"testopia-missing-parameter","success":false}
> 
> I get the same error with the "Environment" select box.
> 
> I also get the same error under "My Cases" for the "Category" field.

I am seeing these errors too, though I'm not sure if this was an issue in Bugzilla 4.4.6 without the patch (I have this patch in place on BNC, and haven't heard this error from any of our heavy Testopia users). I'll have to create these conditions in a test environment and try it out. If it is, I'll create a new bug to fix this and consider it a blocker for a Testopia 2.6 release.

Comment 10

[u430950]

If this issue does not exist in 4.4.6 without the patch, then we can mark the patch '-'

Comment 11

[u430950]

This issue does exist prior to the patch. Assuming everything else is fine with the patch, it should be okay to '+'.

I will create another bug for the issue brought up in comment 8

Comment 12

[Frédéric Buclin]

Comment on attachment 8567143 [details] [diff] [review]
patch.diff

I didn't make sure that every method is listed in PUBLIC_METHODS. I assume you already checked that. r=LpSolit

Comment 13

[Frédéric Buclin]

To ssh://[e-mail]/bugzilla/extensions/Testopia.git
   2f5a32c..d68ed0c  master -> master

Comment 14

[u430950]

(In reply to Frédéric Buclin from comment #12)
> Comment on attachment 8567143 [details] [diff] [review]
> patch.diff
> 
> I didn't make sure that every method is listed in PUBLIC_METHODS. I assume
> you already checked that. r=LpSolit

I made sure all methods were accounted for.