Make the default branch main. (#25)

* Start moving to 'main' default branch.

* Fix tests.

* Make tests use main.

* Fix name of dependency.

* Add more recent versions of PHP to travis config.

* Remove unsupported versions of PHP.

* .gitignore fix.

Author: russtaylor

.gitignore

@@ -1,3 +1,4 @@
 vendor
 composer.lock
 patch.diff
+.phpunit.result.cache

.travis.yml

@@ -1,9 +1,9 @@
 language: php
 
 php:
-  - 5.6
-  - 7.0
-  - 7.1
+  - 7.3
+  - 7.4
+  - 8.0
 
 before_script:
   - composer self-update

README.md

@@ -21,7 +21,7 @@ TryLib currently supports **Freestyle** projects, when your test suite consist o
         -U, --lines-of-context ...  Generate a diff with n lines of context (like git-diff's -U option)
         -s, --staged          Use staged changes only to generate the diff
         -w, --safelist ...    Generate the patch for only the safelisted files
-        -b, --branch ...      Remote branch to diff and try against [master]
+        -b, --branch ...      Remote branch to diff and try against [main]
 
         -c, --show-results    Show final try job results
         -P, --show-progress   Print subtasks progressively as they complete
@@ -152,8 +152,8 @@ More info can be found in the [divide and concur](http://codeascraft.etsy.com/20
 
 Try will work with your branches! The below scenarios are supported:
 
-* You are working on **master**:
-    * You want to try against master
+* You are working on **main**:
+    * You want to try against main
 
             try [options] [subjobs]
     * You want to diff and try against a different branch
@@ -164,13 +164,13 @@ Try will work with your branches! The below scenarios are supported:
     * You want to try against that branch
 
             try [options] [subjobs]
-    * You want to diff and try against a different branch (master for example)
+    * You want to diff and try against a different branch (main for example)
 
-            try --branch=master [options] [subjobs]
+            try --branch=main [options] [subjobs]
 
 * You are working on a **local branch not tracked**
     * If the remote has a branch with the same name, it will be used to generate the diff and try against it
-    * If the remote does not have a branch with the same name, it will use the default remote : **master**
+    * If the remote does not have a branch with the same name, it will use the default remote : **main**
     * You want to diff and try against a specific remote branch
 
             try --branch=my_branch [options] [subjobs]
@@ -181,7 +181,7 @@ Prior to generate the diff, you can configure try to run a list of pre-checks.
 
     $pre_checks = array(
         new TryLib\Precheck\ScriptRunner('/path/to/some/script'),
-        new TryLib\Precheck\GitCopyBehind(array('master')),
+        new TryLib\Precheck\GitCopyBehind(array('main')),
         new TryLib\Precheck\GitCopyAge(48, 96, $remote_branch)
         new TryLib\Precheck\GitReportUntracked(),
     );

composer.json

@@ -18,7 +18,7 @@
      },
     "require-dev": {
         "phpunit/phpunit": ">=5.5.0",
-        "mikey179/vfsStream": "~1"
+        "mikey179/vfsstream": "~1"
     },
     "autoload": {
         "psr-4": {

phpunit.xml

@@ -7,8 +7,7 @@
          convertNoticesToExceptions="true"
          convertWarningsToExceptions="true"
          processIsolation="false"
-         stopOnFailure="false"
-         syntaxCheck="false">
+         stopOnFailure="false">
     <testsuites>
         <testsuite name="TryLib Test Suite">
            <directory suffix=".php">./tests/</directory>

src/TryRunner/Options.php

@@ -36,12 +36,13 @@ final class Options {
     private function __construct() {}  // Do not instantiate.
 
     public static function parse(
-            $argv,
-            $jenkins_server,
-            $default_jenkins_job,
-            $default_jenkins_job_prefix = null,
-            $default_wc_path = null,
-            $default_remote_branch = "master") {
+        $argv,
+        $jenkins_server,
+        $default_jenkins_job,
+        $default_jenkins_job_prefix = null,
+        $default_wc_path = null,
+        $default_remote_branch = "main"
+    ) {
 
         $default_jenkins_job_prefix = $default_jenkins_job_prefix ?: $default_jenkins_job;
         $default_wc_path = $default_wc_path ?: ".";

tests/JenkinsRunner/FreeStyleProjectTest.php

@@ -12,7 +12,7 @@ class FreeStyleProjectTest extends \PHPUnit\Framework\TestCase {
     private $jenkins_runner;
     private $mock_cmd_runner;
 
-    function setUp() {
+    function setUp(): void {
         parent::setUp();
 
         $this->mock_cmd_runner = $this->getMockBuilder('TryLib\CommandRunner')

tests/JenkinsRunner/MasterProjectTest.php

@@ -12,7 +12,7 @@ class MasterProjectTest extends \PHPUnit\Framework\TestCase {
     private $jenkins_runner;
     private $mock_cmd_runner;
 
-    function setUp() {
+    function setUp(): void {
         parent::setUp();
 
         $this->mock_cmd_runner = $this->getMockBuilder('TryLib\CommandRunner')

tests/JenkinsRunnerTest.php

@@ -29,7 +29,7 @@ class JenkinsRunnerTest extends \PHPUnit\Framework\TestCase {
     private $jenkins_runner;
     private $mock_cmd_runner;
 
-    function setUp() {
+    function setUp(): void {
         parent::setUp();
 
         $this->mock_cmd_runner = $this->getMockBuilder('TryLib\CommandRunner')
@@ -49,12 +49,13 @@ function setUp() {
      * @expectedException Exception
      */
     function testInvalidUrl() {
-            $this->jenkins_runner = new TestRunner(
-                'totallyvalid.com/',
-                self::JENKINS_CLI,
-                self::JENKINS_JOB,
-                $this->mock_cmd_runner
-            );
+        $this->expectException(\Exception::class);
+        $this->jenkins_runner = new TestRunner(
+            'totallyvalid.com/',
+            self::JENKINS_CLI,
+            self::JENKINS_JOB,
+            $this->mock_cmd_runner
+        );
     }
 
 

tests/Precheck/GitCopyAgeTest.php

@@ -8,7 +8,7 @@ class GitCopyAgeTest extends \PHPUnit\Framework\TestCase {
 
     private $mock_cmd_runner;
 
-    function setUp() {
+    function setUp(): void {
         parent::setUp();
         $this->mock_cmd_runner = $this->getMockBuilder('TryLib\CommandRunner')
                                       ->getMock();
@@ -78,7 +78,7 @@ function testWorkingCopyPastMaxBlockingAge() {
                               ->method('terminate')
                               ->with($this->stringContains('you working copy is 100 hours old.'));
 
-        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/master');
+        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/main');
     }
 
     function testWorkingCopyPastMaxWarningAge() {
@@ -110,7 +110,7 @@ function testWorkingCopyPastMaxWarningAge() {
         $this->mock_cmd_runner->expects($this->once())
                               ->method('warn');
 
-        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/master');
+        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/main');
     }
 
 
@@ -141,6 +141,6 @@ function testWorkingCopySuccess() {
         $this->mock_cmd_runner->expects($this->never())
                               ->method('warn');
 
-        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/master');
+        $script_runner->check($this->mock_cmd_runner, 'path', 'origin/main');
     }
 }