Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Added negative project import tests #64

Merged
merged 11 commits into from
May 21, 2020

Conversation

IgorYasko121
Copy link
Collaborator

Related Issue

Proposed Changes

  • Added Bazel test projects.
  • Added tests for test cases.

Additional Info

  • Removed unnecessary line "System.out.println(nature);" from BazelNatureTest.

@IgorYasko121 IgorYasko121 added enhancement New feature or request good first issue Good for newcomers labels Apr 15, 2020
@IgorYasko121 IgorYasko121 self-assigned this Apr 15, 2020
@IgorYasko121 IgorYasko121 force-pushed the 59-negative-project-import-tests branch from 998b2cd to 1da8b44 Compare April 15, 2020 11:09
Copy link
Contributor

@beginningineer beginningineer left a comment

Choose a reason for hiding this comment

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

all looks great except a couple of comments.

@IgorYasko121 IgorYasko121 force-pushed the 59-negative-project-import-tests branch from 9de5bcd to c743746 Compare April 21, 2020 05:58
Copy link
Contributor

@beginningineer beginningineer left a comment

Choose a reason for hiding this comment

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

thank a lot!

@SuppressWarnings("restriction")
public class BazelImportTest {
public abstract class BaseBazelImproterTest {
Copy link
Contributor

Choose a reason for hiding this comment

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

typo BaseBazelImpROterTest

@beginningineer beginningineer merged commit 0f66be1 into master May 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla:signed enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add negative Project Import test cases
3 participants