Add basic testing

sangnguyenplus · sangnguyenplus · commit 29aa17a71736 · 2018-10-25T17:14:27.000+07:00
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,15 @@
+; This file is for unifying the coding style for different editors and IDEs.
+; More information at http://editorconfig.org
+
+root = true
+
+[*]
+charset = utf-8
+indent_size = 4
+indent_style = space
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.md]
+trim_trailing_whitespace = false
diff --git a/.gitattributes b/.gitattributes
@@ -0,0 +1,11 @@
+# Path-based git attributes
+# https://www.kernel.org/pub/software/scm/git/docs/gitattributes.html
+
+# Ignore all test and documentation with "export-ignore".
+/.gitattributes     export-ignore
+/.gitignore         export-ignore
+/.travis.yml        export-ignore
+/phpunit.xml.dist   export-ignore
+/.scrutinizer.yml   export-ignore
+/tests              export-ignore
+/.editorconfig      export-ignore
diff --git a/.gitignore b/.gitignore
@@ -1 +1,3 @@
 /.idea
+/vendor
+composer.lock
diff --git a/.styleci.yml b/.styleci.yml
@@ -0,0 +1 @@
+preset: laravel
diff --git a/.travis.yml b/.travis.yml
@@ -0,0 +1,21 @@
+language: php
+php:
+- 7.0
+- 7.1
+- 7.2
+- 7.3
+
+env:
+  matrix:
+  - COMPOSER_FLAGS="--prefer-lowest"
+  - COMPOSER_FLAGS=""
+
+before_script:
+- travis_retry composer self-update
+- travis_retry composer update ${COMPOSER_FLAGS} --no-interaction --prefer-source
+
+script:
+- phpunit --coverage-text --coverage-clover=coverage.clover
+
+after_script:
+- php vendor/bin/ocular code-coverage:upload --format=php-clover coverage.clover
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -0,0 +1,55 @@
+# Contributing
+
+Contributions are **welcome** and will be fully **credited**.
+
+Please read and understand the contribution guide before creating an issue or pull request.
+
+## Etiquette
+
+This project is open source, and as such, the maintainers give their free time to build and maintain the source code
+held within. They make the code freely available in the hope that it will be of use to other developers. It would be
+extremely unfair for them to suffer abuse or anger for their hard work.
+
+Please be considerate towards maintainers when raising issues or presenting pull requests. Let's show the
+world that developers are civilized and selfless people.
+
+It's the duty of the maintainer to ensure that all submissions to the project are of sufficient
+quality to benefit the project. Many developers have different skillsets, strengths, and weaknesses. Respect the maintainer's decision, and do not be upset or abusive if your submission is not used.
+
+## Viability
+
+When requesting or submitting new features, first consider whether it might be useful to others. Open
+source projects are used by many developers, who may have entirely different needs to your own. Think about
+whether or not your feature is likely to be used by other users of the project.
+
+## Procedure
+
+Before filing an issue:
+
+- Attempt to replicate the problem, to ensure that it wasn't a coincidental incident.
+- Check to make sure your feature suggestion isn't already present within the project.
+- Check the pull requests tab to ensure that the bug doesn't have a fix in progress.
+- Check the pull requests tab to ensure that the feature isn't already in progress.
+
+Before submitting a pull request:
+
+- Check the codebase to ensure that your feature doesn't already exist.
+- Check the pull requests to ensure that another person hasn't already submitted the feature or fix.
+
+## Requirements
+
+If the project maintainer has any additional requirements, you will find them listed here.
+
+- **[PSR-2 Coding Standard](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md)** - The easiest way to apply the conventions is to install [PHP Code Sniffer](http://pear.php.net/package/PHP_CodeSniffer).
+
+- **Add tests!** - Your patch won't be accepted if it doesn't have tests.
+
+- **Document any change in behaviour** - Make sure the `README.md` and any other relevant documentation are kept up-to-date.
+
+- **Consider our release cycle** - We try to follow [SemVer v2.0.0](http://semver.org/). Randomly breaking public APIs is not an option.
+
+- **One pull request per feature** - If you want to do more than one thing, send multiple pull requests.
+
+- **Send coherent history** - Make sure each individual commit in your pull request is meaningful. If you had to make multiple intermediate commits while developing, please [squash them](http://www.git-scm.com/book/en/v2/Git-Tools-Rewriting-History#Changing-Multiple-Commit-Messages) before submitting.
+
+**Happy coding**!
diff --git a/README.md b/README.md
@@ -1,9 +1,11 @@
 # Laravel Assets management
 
-![Total downloads](https://poser.pugx.org/botble/assets/d/total.svg)
-![Latest Stable Version](https://poser.pugx.org/botble/assets/v/stable.svg)
-![Latest Unstable Version](https://poser.pugx.org/botble/assets/v/unstable)
-![License](https://poser.pugx.org/botble/assets/license)
+[![Latest Version](https://img.shields.io/github/release/botble/laravel-assets.svg?style=flat-square)](https://github.com/botble/laravel-assets/releases)
+[![Software License](https://img.shields.io/badge/license-MIT-brightgreen.svg?style=flat-square)](LICENSE.md)
+[![Build Status](https://img.shields.io/travis/botble/assets/master.svg?style=flat-square)](https://travis-ci.org/botble/assets)
+[![Quality Score](https://img.shields.io/scrutinizer/g/botble/assets.svg?style=flat-square)](https://scrutinizer-ci.com/g/botble/assets)
+[![StyleCI](https://styleci.io/repos/32067087/shield)](https://styleci.io/repos/32067087)
+[![Total Downloads](https://img.shields.io/packagist/dt/botble/assets.svg?style=flat-square)](https://packagist.org/packages/botble/assets)
 
 ## Installation
 
diff --git a/composer.json b/composer.json
@@ -16,13 +16,24 @@
         }
     ],
     "require": {
-        "laravelcollective/html": "5.1.*|5.2.*|5.3.*|5.4.*|5.5.*|5.6.*|5.7.*"
+        "php": "^7.0",
+        "laravel/framework": "~5.1",
+        "laravelcollective/html": "~5.1"
+    },
+    "require-dev": {
+        "phpunit/phpunit": "^7.4",
+        "mockery/mockery": "^1.0"
     },
     "autoload": {
         "psr-4": {
             "Botble\\Assets\\": "src"
         }
     },
+    "autoload-dev": {
+        "psr-4": {
+            "Botble\\Assets\\Tests\\": "tests"
+        }
+    },
     "extra": {
         "laravel": {
             "providers": [
@@ -33,5 +44,8 @@
             }
         }
     },
+    "config": {
+        "sort-packages": true
+    },
     "minimum-stability": "stable"
-}
+}
diff --git a/phpunit.xml b/phpunit.xml
@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit bootstrap="vendor/autoload.php"
+         backupGlobals="false"
+         backupStaticAttributes="false"
+         colors="true"
+         verbose="true"
+         convertErrorsToExceptions="true"
+         convertNoticesToExceptions="true"
+         convertWarningsToExceptions="true"
+         processIsolation="false"
+         stopOnFailure="false">
+    <testsuites>
+        <testsuite name="Botble Test Suite">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+    <filter>
+        <whitelist>
+            <directory suffix=".php">src/</directory>
+        </whitelist>
+    </filter>
+</phpunit>
diff --git a/scrutinizer.yml b/scrutinizer.yml
@@ -0,0 +1,18 @@
+filter:
+  excluded_paths: [tests/*]
+
+checks:
+  php:
+    remove_extra_empty_lines: true
+    remove_php_closing_tag: true
+    remove_trailing_whitespace: true
+    fix_use_statements:
+      remove_unused: true
+      preserve_multiple: false
+      preserve_blanklines: true
+      order_alphabetically: true
+    fix_php_opening_tag: true
+    fix_linefeed: true
+    fix_line_ending: true
+    fix_identation_4spaces: true
+    fix_doc_comments: true
diff --git a/tests/Unit/AssetsTest.php b/tests/Unit/AssetsTest.php
@@ -0,0 +1,64 @@
+<?php
+
+namespace Botble\Assets\Tests;
+
+use PHPUnit\Framework\TestCase;
+use Mockery;
+use Botble\Assets\Assets;
+
+class AssetsTest extends TestCase
+{
+    /**
+     * @var Mockery\Mock|Assets
+     */
+    protected $assets;
+
+    public function setUp()
+    {
+        $this->assets = Mockery::mock(Assets::class);
+    }
+
+    public function tearDown()
+    {
+        Mockery::close();
+    }
+
+    public function testGetJavascript()
+    {
+        $this->assets
+            ->shouldReceive('getJavascript')
+            ->once()
+            ->andReturn([
+                [
+                    'src'        => '/js/app.js?v=1.0',
+                    'attributes' => [],
+                ],
+            ]);
+
+        $response = $this->assets->getJavascript();
+
+        $this->assertNotEmpty($response);
+        $this->assertEmpty(array_get(array_first($response), 'attributes'));
+    }
+
+    public function testGetStylesheets()
+    {
+        $this->assets
+            ->shouldReceive('getStylesheets')
+            ->once()
+            ->andReturn([
+                [
+                    'src'        => '/css/bootstrap.css?v=1.0',
+                    'attributes' => [
+                        'integrity' => 'sha384-WskhaSGFgHYWDcbwN70/dfYBj47jz9qbsMId/iRN3ewGhXQFZCSftd1LZCfmhktB',
+                        'crossorigin' => 'anonymous',
+                    ],
+                ],
+            ]);
+
+        $response = $this->assets->getStylesheets();
+
+        $this->assertNotEmpty($response);
+        $this->assertNotEmpty(array_get(array_first($response), 'attributes'));
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -1 +1,3 @@`
`1`	`1`	`/.idea`
	`2`	`+/vendor`
	`3`	`+composer.lock`