ladybirdweb / import-export

tests/ImportTest.php

Indentation +100 added lines, -100 removed lines

@@ -15,115 +15,115 @@
 
 class ImportTest extends TestCase
 {
-	use RefreshDatabase;
+    use RefreshDatabase;
 
-	protected $import;
+    protected $import;
 
-	protected function getPackageProviders($app)
-	{
-	    return ['Ladybirdweb\ImportExport\ImportExportServiceProvider'];
-	}
+    protected function getPackageProviders($app)
+    {
+        return ['Ladybirdweb\ImportExport\ImportExportServiceProvider'];
+    }
 
     protected function getEnvironmentSetUp($app)
     {
         $app['config']->set('database.default', 'testing');
     }
 
-	protected function setUp ()
-	{
-	    parent::setUp();
+    protected function setUp ()
+    {
+        parent::setUp();
 
-	    $this->artisan('migrate', ['--database' => 'testing']);
+        $this->artisan('migrate', ['--database' => 'testing']);
 
-	    Route::middleware('web')->group(function() {
+        Route::middleware('web')->group(function() {
 		    
-			Route::get( '/import/{id}/progress', [ 'as' => 'ladybirdweb.import.ajax.progress', 'uses' => 'Ladybirdweb\ImportExport\Import@returnImportProgress']);
-
-		});
-
-		Storage::putFileAs('imports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.csv');
-
-		$this->import = ModelImport::create([
-			'file' => 'imports/test.csv',
-			'file_rows' => 104,
-			'db_cols' => [ 'name', 'email', 'password'],
-			'model_map' => ['email', 'name', 'password']
-		]);
-	}
-
-	/**
-	* @test
-	*/
-	public function create_new_import_success()
-	{
-		$import = Import::createImport('imports/test.csv', [ 'name', 'email', 'password']);
-
-		$this->assertInstanceOf(ModelImport::class, $import);
-	}
-
-	/**
-	* @test
-	*/
-	public function fetch_import()
-	{
-		$import = Import::getImport($this->import->id);
-
-		$this->assertInstanceOf(ModelImport::class, $import);
-	}
-
-	/**
-	* @test
-	*/
-	public function get_few_rows_from_uploaded_file()
-	{
-		$csv_data = Import::getImportFileData($this->import->id);
-
-		$this->assertInternalType( 'array', $csv_data );
-	}
-
-	/**
-	* @test
-	*/
-	public function store_data_map_with_csv_cols()
-	{
-		$import = Import::setDataMap(['email', 'name', 'password'], $this->import->id);
-
-		$this->assertInstanceOf(ModelImport::class, $import);
-	}
-
-	/**
-	* @test
-	*/
-	public function sucess_to_dispatch_given_job_class()
-	{
-		$import = $id = $this->import;
-
-		Queue::fake();
-
-		Import::dispatchImportJob( FakeJob::class, $import );
-
-		Queue::assertPushedOn('importing', FakeJob::class);
-	}
-
-	/**
-	* @test
-	*/
-	public function check_import_progress()
-	{
-		$response = $this->json( 'GET', '/import/1/progress' );
-
-		$response->assertStatus(200);
-
-		$response->assertJsonFragment( ['status' => 200] );
-
-		$response->assertJsonFragment( ['progress'] );
-	}
-
-	/**
-	* @test
-	*/
-	public function remove_import()
-	{
-		$this->assertTrue(Import::removeImport($this->import->id));
-	}
+            Route::get( '/import/{id}/progress', [ 'as' => 'ladybirdweb.import.ajax.progress', 'uses' => 'Ladybirdweb\ImportExport\Import@returnImportProgress']);
+
+        });
+
+        Storage::putFileAs('imports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.csv');
+
+        $this->import = ModelImport::create([
+            'file' => 'imports/test.csv',
+            'file_rows' => 104,
+            'db_cols' => [ 'name', 'email', 'password'],
+            'model_map' => ['email', 'name', 'password']
+        ]);
+    }
+
+    /**
+     * @test
+     */
+    public function create_new_import_success()
+    {
+        $import = Import::createImport('imports/test.csv', [ 'name', 'email', 'password']);
+
+        $this->assertInstanceOf(ModelImport::class, $import);
+    }
+
+    /**
+     * @test
+     */
+    public function fetch_import()
+    {
+        $import = Import::getImport($this->import->id);
+
+        $this->assertInstanceOf(ModelImport::class, $import);
+    }
+
+    /**
+     * @test
+     */
+    public function get_few_rows_from_uploaded_file()
+    {
+        $csv_data = Import::getImportFileData($this->import->id);
+
+        $this->assertInternalType( 'array', $csv_data );
+    }
+
+    /**
+     * @test
+     */
+    public function store_data_map_with_csv_cols()
+    {
+        $import = Import::setDataMap(['email', 'name', 'password'], $this->import->id);
+
+        $this->assertInstanceOf(ModelImport::class, $import);
+    }
+
+    /**
+     * @test
+     */
+    public function sucess_to_dispatch_given_job_class()
+    {
+        $import = $id = $this->import;
+
+        Queue::fake();
+
+        Import::dispatchImportJob( FakeJob::class, $import );
+
+        Queue::assertPushedOn('importing', FakeJob::class);
+    }
+
+    /**
+     * @test
+     */
+    public function check_import_progress()
+    {
+        $response = $this->json( 'GET', '/import/1/progress' );
+
+        $response->assertStatus(200);
+
+        $response->assertJsonFragment( ['status' => 200] );
+
+        $response->assertJsonFragment( ['progress'] );
+    }
+
+    /**
+     * @test
+     */
+    public function remove_import()
+    {
+        $this->assertTrue(Import::removeImport($this->import->id));
+    }
 }

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
         $app['config']->set('database.default', 'testing');
     }
 
-	protected function setUp ()
+	protected function setUp()
 	{
 	    parent::setUp();
 
@@ -37,16 +37,16 @@ 
 
 	    Route::middleware('web')->group(function() {
 		    
-			Route::get( '/import/{id}/progress', [ 'as' => 'ladybirdweb.import.ajax.progress', 'uses' => 'Ladybirdweb\ImportExport\Import@returnImportProgress']);
+			Route::get('/import/{id}/progress', ['as' => 'ladybirdweb.import.ajax.progress', 'uses' => 'Ladybirdweb\ImportExport\Import@returnImportProgress']);
 
 		});
 
-		Storage::putFileAs('imports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.csv');
+		Storage::putFileAs('imports', new File(__DIR__.'/storage/test/test.csv'), 'test.csv');
 
 		$this->import = ModelImport::create([
 			'file' => 'imports/test.csv',
 			'file_rows' => 104,
-			'db_cols' => [ 'name', 'email', 'password'],
+			'db_cols' => ['name', 'email', 'password'],
 			'model_map' => ['email', 'name', 'password']
 		]);
 	}
@@ -56,7 +56,7 @@ 
 	*/
 	public function create_new_import_success()
 	{
-		$import = Import::createImport('imports/test.csv', [ 'name', 'email', 'password']);
+		$import = Import::createImport('imports/test.csv', ['name', 'email', 'password']);
 
 		$this->assertInstanceOf(ModelImport::class, $import);
 	}
@@ -78,7 +78,7 @@ 
 	{
 		$csv_data = Import::getImportFileData($this->import->id);
 
-		$this->assertInternalType( 'array', $csv_data );
+		$this->assertInternalType('array', $csv_data);
 	}
 
 	/**
@@ -100,7 +100,7 @@ 
 
 		Queue::fake();
 
-		Import::dispatchImportJob( FakeJob::class, $import );
+		Import::dispatchImportJob(FakeJob::class, $import);
 
 		Queue::assertPushedOn('importing', FakeJob::class);
 	}
@@ -110,13 +110,13 @@ 
 	*/
 	public function check_import_progress()
 	{
-		$response = $this->json( 'GET', '/import/1/progress' );
+		$response = $this->json('GET', '/import/1/progress');
 
 		$response->assertStatus(200);
 
-		$response->assertJsonFragment( ['status' => 200] );
+		$response->assertJsonFragment(['status' => 200]);
 
-		$response->assertJsonFragment( ['progress'] );
+		$response->assertJsonFragment(['progress']);
 	}
 
 	/**

tests/ExportTest.php

Indentation +67 added lines, -67 removed lines

@@ -16,102 +16,102 @@
 
 class ExportTest extends TestCase
 {
-	use RefreshDatabase;
+    use RefreshDatabase;
 
-	protected $export;
+    protected $export;
 
-	protected function getPackageProviders($app)
-	{
-	    return ['Ladybirdweb\ImportExport\ImportExportServiceProvider'];
-	}
+    protected function getPackageProviders($app)
+    {
+        return ['Ladybirdweb\ImportExport\ImportExportServiceProvider'];
+    }
 
     protected function getEnvironmentSetUp($app)
     {
         $app['config']->set('database.default', 'testing');
     }
 
-	protected function setUp ()
-	{
-	    parent::setUp();
+    protected function setUp ()
+    {
+        parent::setUp();
 
-	    $this->artisan('migrate', ['--database' => 'testing']);
+        $this->artisan('migrate', ['--database' => 'testing']);
 
-	    Route::middleware('web')->group(function() {
+        Route::middleware('web')->group(function() {
 
-			Route::get('/ticket/export/{id}', [ 'as' => 'ticket.export.progress', 'uses' => 'Ladybirdweb\ImportExport\Export@showExportStatus']);
+            Route::get('/ticket/export/{id}', [ 'as' => 'ticket.export.progress', 'uses' => 'Ladybirdweb\ImportExport\Export@showExportStatus']);
 
-			Route::get( '/export/{id}/download',  [ 'as' => 'ladybirdweb.export.download', 'uses' => 'Ladybirdweb\ImportExport\Export@downloadExportedFile']);
+            Route::get( '/export/{id}/download',  [ 'as' => 'ladybirdweb.export.download', 'uses' => 'Ladybirdweb\ImportExport\Export@downloadExportedFile']);
 
-		});
+        });
 
-		Storage::putFileAs('exports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.xls');
+        Storage::putFileAs('exports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.xls');
 
-		$this->export = ModelExport::create([
-			'file' => 'test.xls',
-			'query' => \App\Models\User::select([ 'name', 'email', 'created_at' ])->getModel(),
-			'type' => 'xls'
-		]);
-	}
+        $this->export = ModelExport::create([
+            'file' => 'test.xls',
+            'query' => \App\Models\User::select([ 'name', 'email', 'created_at' ])->getModel(),
+            'type' => 'xls'
+        ]);
+    }
 
-	/**
-	* @test
-	*/
-	public function data_export_initiated_and_dispatched()
-	{
-		Queue::fake();
+    /**
+     * @test
+     */
+    public function data_export_initiated_and_dispatched()
+    {
+        Queue::fake();
 
-		$export = Export::export( \App\Models\User::select([ 'name', 'email', 'created_at' ]), 'xls' );
+        $export = Export::export( \App\Models\User::select([ 'name', 'email', 'created_at' ]), 'xls' );
 
-		$this->assertInstanceOf( ModelExport::class, $export );
+        $this->assertInstanceOf( ModelExport::class, $export );
 
-		Queue::assertPushedOn('exporting', ExportJob::class);
-	}
+        Queue::assertPushedOn('exporting', ExportJob::class);
+    }
 
-	/**
-	* @test
-	*/
-	public function see_export_progress_page()
-	{
-		$response =  $this->get('/ticket/export/' . $this->export->id);
+    /**
+     * @test
+     */
+    public function see_export_progress_page()
+    {
+        $response =  $this->get('/ticket/export/' . $this->export->id);
 
-		$response->assertStatus(200);
+        $response->assertStatus(200);
 
-		$response->assertSee('Export');
-	}
+        $response->assertSee('Export');
+    }
 
-	/**
-	* @test
-	*/
-	public function check_export_progress_ajax()
-	{
-		$response = $this->json( 'GET', '/export/' . $this->export->id . '/progress');
+    /**
+     * @test
+     */
+    public function check_export_progress_ajax()
+    {
+        $response = $this->json( 'GET', '/export/' . $this->export->id . '/progress');
 
-		$response->assertStatus(200);
+        $response->assertStatus(200);
 
-		$response->assertJsonFragment( ['status' => 200] );
+        $response->assertJsonFragment( ['status' => 200] );
 
-		$response->assertJsonFragment( ['progress'] );
-	}
+        $response->assertJsonFragment( ['progress'] );
+    }
 
-	/**
-	* @test
-	*/
-	public function try_download_exported_file()
-	{
-		$response = $this->get('/export/' . $this->export->id . '/download');
+    /**
+     * @test
+     */
+    public function try_download_exported_file()
+    {
+        $response = $this->get('/export/' . $this->export->id . '/download');
 
-		$response->assertHeader( 'content-disposition', 'attachment; filename="test.xls"');
+        $response->assertHeader( 'content-disposition', 'attachment; filename="test.xls"');
 
-		$response->assertStatus(200);
-	}
+        $response->assertStatus(200);
+    }
 
-	/**
-	* @test
-	*/
-	public function fail_download_exported_file()
-	{
-		$response = $this->get('/export/987654321/download');
+    /**
+     * @test
+     */
+    public function fail_download_exported_file()
+    {
+        $response = $this->get('/export/987654321/download');
 
-		$response->assertStatus(404);
-	}
+        $response->assertStatus(404);
+    }
 }

Spacing +13 added lines, -13 removed lines

@@ -30,7 +30,7 @@ 
         $app['config']->set('database.default', 'testing');
     }
 
-	protected function setUp ()
+	protected function setUp()
 	{
 	    parent::setUp();
 
@@ -38,17 +38,17 @@ 
 
 	    Route::middleware('web')->group(function() {
 
-			Route::get('/ticket/export/{id}', [ 'as' => 'ticket.export.progress', 'uses' => 'Ladybirdweb\ImportExport\Export@showExportStatus']);
+			Route::get('/ticket/export/{id}', ['as' => 'ticket.export.progress', 'uses' => 'Ladybirdweb\ImportExport\Export@showExportStatus']);
 
-			Route::get( '/export/{id}/download',  [ 'as' => 'ladybirdweb.export.download', 'uses' => 'Ladybirdweb\ImportExport\Export@downloadExportedFile']);
+			Route::get('/export/{id}/download', ['as' => 'ladybirdweb.export.download', 'uses' => 'Ladybirdweb\ImportExport\Export@downloadExportedFile']);
 
 		});
 
-		Storage::putFileAs('exports', new File( __DIR__ . '/storage/test/test.csv' ), 'test.xls');
+		Storage::putFileAs('exports', new File(__DIR__.'/storage/test/test.csv'), 'test.xls');
 
 		$this->export = ModelExport::create([
 			'file' => 'test.xls',
-			'query' => \App\Models\User::select([ 'name', 'email', 'created_at' ])->getModel(),
+			'query' => \App\Models\User::select(['name', 'email', 'created_at'])->getModel(),
 			'type' => 'xls'
 		]);
 	}
@@ -60,9 +60,9 @@ 
 	{
 		Queue::fake();
 
-		$export = Export::export( \App\Models\User::select([ 'name', 'email', 'created_at' ]), 'xls' );
+		$export = Export::export(\App\Models\User::select(['name', 'email', 'created_at']), 'xls');
 
-		$this->assertInstanceOf( ModelExport::class, $export );
+		$this->assertInstanceOf(ModelExport::class, $export);
 
 		Queue::assertPushedOn('exporting', ExportJob::class);
 	}
@@ -72,7 +72,7 @@ 
 	*/
 	public function see_export_progress_page()
 	{
-		$response =  $this->get('/ticket/export/' . $this->export->id);
+		$response = $this->get('/ticket/export/'.$this->export->id);
 
 		$response->assertStatus(200);
 
@@ -84,13 +84,13 @@ 
 	*/
 	public function check_export_progress_ajax()
 	{
-		$response = $this->json( 'GET', '/export/' . $this->export->id . '/progress');
+		$response = $this->json('GET', '/export/'.$this->export->id.'/progress');
 
 		$response->assertStatus(200);
 
-		$response->assertJsonFragment( ['status' => 200] );
+		$response->assertJsonFragment(['status' => 200]);
 
-		$response->assertJsonFragment( ['progress'] );
+		$response->assertJsonFragment(['progress']);
 	}
 
 	/**
@@ -98,9 +98,9 @@ 
 	*/
 	public function try_download_exported_file()
 	{
-		$response = $this->get('/export/' . $this->export->id . '/download');
+		$response = $this->get('/export/'.$this->export->id.'/download');
 
-		$response->assertHeader( 'content-disposition', 'attachment; filename="test.xls"');
+		$response->assertHeader('content-disposition', 'attachment; filename="test.xls"');
 
 		$response->assertStatus(200);
 	}