ladybirdweb / import-export

tests/ImportTest.php

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( storage_path( 'test/test.csv' ) ), 'test.csv');
+		Storage::putFileAs('imports', new File(storage_path('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

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( storage_path( 'test/test.csv' ) ), 'test.xls');
+		Storage::putFileAs('exports', new File(storage_path('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);
 	}

tests/ImportExportLogTest.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
         $app['config']->set('database.default', 'testing');
     }
 
-	protected function setUp ()
+	protected function setUp()
 	{
 	    parent::setUp();
 
@@ -34,7 +34,7 @@ 
 		$this->import = Import::create([
 			'file' => 'imports/import-1530262997.csv',
 			'file_rows' => 104,
-			'db_cols' => [ 'name', 'email', 'password'],
+			'db_cols' => ['name', 'email', 'password'],
 			'model_map' => ['email', 'name', 'password']
 		]);
 	}
@@ -46,9 +46,9 @@ 
 	{
 		$import = $this->import;
 
-		$result = ImportExportLog::logImportError( $import, ['data' => 'this is test data'], 'This is not expected' );
+		$result = ImportExportLog::logImportError($import, ['data' => 'this is test data'], 'This is not expected');
 
-		$this->assertInstanceOf( ModelImportExportLog::class, $result );
+		$this->assertInstanceOf(ModelImportExportLog::class, $result);
 	}
 
 	/**
@@ -58,14 +58,14 @@ 
 	{
 		$import = $this->import;
 
-		$log = ImportExportLog::logImportError( $import, ['data' => 'this is test data'], 'This is not expected' );
+		$log = ImportExportLog::logImportError($import, ['data' => 'this is test data'], 'This is not expected');
 
-		$log_in_db = ImportExportLog::getLogs( $log->id );
+		$log_in_db = ImportExportLog::getLogs($log->id);
 
-		$this->assertInternalType( 'array', $log_in_db );
+		$this->assertInternalType('array', $log_in_db);
 
-		$this->assertArrayHasKey( 'data', $log_in_db[0] );
+		$this->assertArrayHasKey('data', $log_in_db[0]);
 
-		$this->assertArrayHasKey( 'message', $log_in_db[0] );
+		$this->assertArrayHasKey('message', $log_in_db[0]);
 	}
 }