Skip to content

Conversation

carycheng
Copy link

No description provided.

@boxcla
Copy link

boxcla commented Feb 13, 2018

Hi @carycheng, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

@carycheng carycheng changed the title Java test automation - PLEASE DO NOT MERGE Java test automation Apr 30, 2018
build.gradle Outdated
task integrationTest(type: Test) {
description 'Runs the integration tests.'
group 'Verification'
systemProperty "USE_REAL_API", System.getProperty("USE_REAL_API")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be removed

@@ -1,13 +0,0 @@
# Remove the .template extension to this file to configure tests. The config.properties file should never be added to

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file shouldn't be deleted

import com.github.tomakehurst.wiremock.client.WireMock;
import com.github.tomakehurst.wiremock.client.WireMock.*;

import javax.swing.*;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably don't need to import this

@carycheng carycheng merged commit 1cea15f into master May 3, 2018
@mattwiller mattwiller deleted the java_test_automation branch December 15, 2018 00:45
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

Successfully merging this pull request may close these issues.

3 participants