My testing environment is ready (re-hosted the Omni Converter service locally, cURL and POSTman to execute test requests, Eclipse to view source code, a text file for bugs), so it's time to launch back into exploratory testing.
Below is a log of my test session. It's a bit light on detail, mostly because the tests were so short (ie, executing one HTTP request and viewing the response) and numerous that they didn't feel worth fully documenting. In hindsight this was a mistake, since I need to reproduce those details in my bug reports anyways. I should also take a look online for popular exploratory log formats/guidelines; this one was off the top of my head, and I'm sure there are a thousand improvements to be made.
-Tried to call API via browser, received "not authorized" and was asked for credentials. Searched through the source code and found that the parameters -p and -e (which are required when starting the OCC service) specify your password and username respectively. Gave those values as my credentials, and was successfully validated. Apparently the executable also suports a --help parameter, which would've told me about that (and the other parameters).
-Testing the /v1/user route. It's is supposed to count the number of api calls a user has made total, per day, and per minute. I am attempting to reproduce the null-values issue with the counters that I saw when using Palantir's browser frontend to access their instance of OCC.
ISSUE 1: Calling /v1/user, the calls-counter per-day sometimes resets to 1 along with the per-minute counter. It seems to happen more often when the service has just started, or when the calls-per-day counter is low.
FOLLOWUP ISSUE 1: After searching through the source code, it appears this happens because of the following line in ApiServer.java:
scheduler.scheduleAtFixedRate(ApiUtils.resetDayHits, 1, 14400, TimeUnit.MINUTES);
The second argument, 1, means that resetDayHits will fire 1 minute after the OCC service starts up before it goes into its daily schedule. So, any API calls in that first minute will be lost from the daily counter. That parameter should be changed to either 0 or 14400.
-Retrieved example DAGs in various formats, using requests like:
Which returns the download URL for the specified DAG:
ISSUE 2: The download URL doesn't require authentication. This is inconsisitent with the requirement for authentication to see the list of DAGs, and to retrieve the access link for a particular DAG.
CLOSED ISSUE 2: According to documentation, this is correct behaviour.
ISSUE 3: The key at the end of the download link (ie. uWq1l5PSvRc4) can be modified to slightly change the DAG which is recieved. Nodes are relabelled randomly, and edges and node contents are modified to fit the new labels. For example, node 0 with contents "AAA" and edge [0, 2] might become node 1 with contents "AAA" and edge [1, 2]. Happens with both replacement and newly-created DAGs.
-Uploaded a new DAG, which was a copy of the example JSON DAG provided in Palantir's source. Needed paramters "format JSON" and "file [file]".
-Replaced the contents of a DAG (using POST /v1/documents/[id])
-Redownloaded the uploaded DAGs.
FOLLOWUP ISSUE 3: May not be related to changes in the key. Repeated calls to the same download URL without modifying the key also results in randomly relabeled nodes.
-Deleted single DAG
-Uploaded new DAG without specifying an ID number
ISSUE 4: New DAG was not assigned the ID number of the previously deleted one.
CLOSED ISSUE 4: Not necessarily a bug. The problem is that eventually ID numbers become unmanagably high, but that won't hapen for a long time. Tracking and reusing deleted ID's would be more costly.
-Deleted all DAGs
-Uploaded new DAGs. Interesting note, ID-numbering did restart from 1 after deleting all, which is good.