Uploaded image for project: 'Blazegraph (by SYSTAP)'
  1. Blazegraph (by SYSTAP)
  2. BLZG-1461

Add Unit Tests to Blazegraph-samples projects

    Details

    • Type: Task
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: BLAZEGRAPH_RELEASE_1_5_3
    • Component/s: None
    • Labels:
      None

      Description

      Please add unit tests to each of the blazegraph-samples artifacts to validate that blazegraph version changes have not inadvertently broken any of the samples.

        Activity

        Hide
        maria.krokhaleva maria.krokhaleva added a comment -

        We are going to add tests to the blazegraph-samples with dependencies on 1.5.3 version due to changes in project structure and packaging.
        Tests will ensure following:
        1. Syntax consistency of sample applications with blazegraph core artifacts.
        2. No runtime errors while executing main(). Local tests will use journals created in target directory, remote tests will setup NanoSparqlServer instances, which will also store their files in target directory.

        Do you have any comments/concerns?

        Show
        maria.krokhaleva maria.krokhaleva added a comment - We are going to add tests to the blazegraph-samples with dependencies on 1.5.3 version due to changes in project structure and packaging. Tests will ensure following: 1. Syntax consistency of sample applications with blazegraph core artifacts. 2. No runtime errors while executing main(). Local tests will use journals created in target directory, remote tests will setup NanoSparqlServer instances, which will also store their files in target directory. Do you have any comments/concerns?
        Hide
        beebs Brad Bebee added a comment -

        Yes, that works. Please try to re-use existing unit tests that run the embedded NSS.

        Show
        beebs Brad Bebee added a comment - Yes, that works. Please try to re-use existing unit tests that run the embedded NSS.
        Hide
        beebs Brad Bebee added a comment -

        igorkim Can this one be completed this week?

        Show
        beebs Brad Bebee added a comment - igorkim Can this one be completed this week?
        Hide
        igorkim igorkim added a comment -

        Brad Bebee, yes, we are going to create pull request in a few days. It will also update version of blazegraph dependencies to master (1.5.3).

        Show
        igorkim igorkim added a comment - Brad Bebee , yes, we are going to create pull request in a few days. It will also update version of blazegraph dependencies to master (1.5.3).
        Hide
        beebs Brad Bebee added a comment -

        Let's use this ticket for the PR on the unit tests and validate that it works against the 1.5.3 snapshot. I created https://jira.blazegraph.com/browse/BLZG-1492 as a ticket for the samples release and update to the 1.5.3 release.

        Show
        beebs Brad Bebee added a comment - Let's use this ticket for the PR on the unit tests and validate that it works against the 1.5.3 snapshot. I created https://jira.blazegraph.com/browse/BLZG-1492 as a ticket for the samples release and update to the 1.5.3 release.

          People

          • Assignee:
            maria.krokhaleva maria.krokhaleva
            Reporter:
            beebs Brad Bebee
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: