Skip to content

Commit f082249

Browse files
committed
rm slop
1 parent e560b57 commit f082249

1 file changed

Lines changed: 2 additions & 111 deletions

File tree

cmd/browsers.go

Lines changed: 2 additions & 111 deletions
Original file line numberDiff line numberDiff line change
@@ -311,10 +311,6 @@ func (b BrowsersCmd) LogsStream(ctx context.Context, in BrowsersLogsStreamInput)
311311
pterm.Error.Println("logs service not available")
312312
return nil
313313
}
314-
if b.browsers == nil {
315-
pterm.Error.Println("browsers service not available")
316-
return nil
317-
}
318314
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
319315
if err != nil {
320316
return util.CleanedUpSdkError{Err: err}
@@ -372,14 +368,6 @@ type BrowsersReplaysDownloadInput struct {
372368
}
373369

374370
func (b BrowsersCmd) ReplaysList(ctx context.Context, in BrowsersReplaysListInput) error {
375-
if b.replays == nil {
376-
pterm.Error.Println("replays service not available")
377-
return nil
378-
}
379-
if b.browsers == nil {
380-
pterm.Error.Println("browsers service not available")
381-
return nil
382-
}
383371
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
384372
if err != nil {
385373
return util.CleanedUpSdkError{Err: err}
@@ -405,14 +393,6 @@ func (b BrowsersCmd) ReplaysList(ctx context.Context, in BrowsersReplaysListInpu
405393
}
406394

407395
func (b BrowsersCmd) ReplaysStart(ctx context.Context, in BrowsersReplaysStartInput) error {
408-
if b.replays == nil {
409-
pterm.Error.Println("replays service not available")
410-
return nil
411-
}
412-
if b.browsers == nil {
413-
pterm.Error.Println("browsers service not available")
414-
return nil
415-
}
416396
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
417397
if err != nil {
418398
return util.CleanedUpSdkError{Err: err}
@@ -438,14 +418,6 @@ func (b BrowsersCmd) ReplaysStart(ctx context.Context, in BrowsersReplaysStartIn
438418
}
439419

440420
func (b BrowsersCmd) ReplaysStop(ctx context.Context, in BrowsersReplaysStopInput) error {
441-
if b.replays == nil {
442-
pterm.Error.Println("replays service not available")
443-
return nil
444-
}
445-
if b.browsers == nil {
446-
pterm.Error.Println("browsers service not available")
447-
return nil
448-
}
449421
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
450422
if err != nil {
451423
return util.CleanedUpSdkError{Err: err}
@@ -463,14 +435,6 @@ func (b BrowsersCmd) ReplaysStop(ctx context.Context, in BrowsersReplaysStopInpu
463435
}
464436

465437
func (b BrowsersCmd) ReplaysDownload(ctx context.Context, in BrowsersReplaysDownloadInput) error {
466-
if b.replays == nil {
467-
pterm.Error.Println("replays service not available")
468-
return nil
469-
}
470-
if b.browsers == nil {
471-
pterm.Error.Println("browsers service not available")
472-
return nil
473-
}
474438
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
475439
if err != nil {
476440
return util.CleanedUpSdkError{Err: err}
@@ -543,10 +507,6 @@ func (b BrowsersCmd) ProcessExec(ctx context.Context, in BrowsersProcessExecInpu
543507
pterm.Error.Println("process service not available")
544508
return nil
545509
}
546-
if b.browsers == nil {
547-
pterm.Error.Println("browsers service not available")
548-
return nil
549-
}
550510
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
551511
if err != nil {
552512
return util.CleanedUpSdkError{Err: err}
@@ -609,10 +569,6 @@ func (b BrowsersCmd) ProcessSpawn(ctx context.Context, in BrowsersProcessSpawnIn
609569
pterm.Error.Println("process service not available")
610570
return nil
611571
}
612-
if b.browsers == nil {
613-
pterm.Error.Println("browsers service not available")
614-
return nil
615-
}
616572
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
617573
if err != nil {
618574
return util.CleanedUpSdkError{Err: err}
@@ -651,10 +607,6 @@ func (b BrowsersCmd) ProcessKill(ctx context.Context, in BrowsersProcessKillInpu
651607
pterm.Error.Println("process service not available")
652608
return nil
653609
}
654-
if b.browsers == nil {
655-
pterm.Error.Println("browsers service not available")
656-
return nil
657-
}
658610
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
659611
if err != nil {
660612
return util.CleanedUpSdkError{Err: err}
@@ -677,10 +629,6 @@ func (b BrowsersCmd) ProcessStatus(ctx context.Context, in BrowsersProcessStatus
677629
pterm.Error.Println("process service not available")
678630
return nil
679631
}
680-
if b.browsers == nil {
681-
pterm.Error.Println("browsers service not available")
682-
return nil
683-
}
684632
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
685633
if err != nil {
686634
return util.CleanedUpSdkError{Err: err}
@@ -703,10 +651,6 @@ func (b BrowsersCmd) ProcessStdin(ctx context.Context, in BrowsersProcessStdinIn
703651
pterm.Error.Println("process service not available")
704652
return nil
705653
}
706-
if b.browsers == nil {
707-
pterm.Error.Println("browsers service not available")
708-
return nil
709-
}
710654
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
711655
if err != nil {
712656
return util.CleanedUpSdkError{Err: err}
@@ -728,10 +672,6 @@ func (b BrowsersCmd) ProcessStdoutStream(ctx context.Context, in BrowsersProcess
728672
pterm.Error.Println("process service not available")
729673
return nil
730674
}
731-
if b.browsers == nil {
732-
pterm.Error.Println("browsers service not available")
733-
return nil
734-
}
735675
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
736676
if err != nil {
737677
return util.CleanedUpSdkError{Err: err}
@@ -847,10 +787,6 @@ func (b BrowsersCmd) FSNewDirectory(ctx context.Context, in BrowsersFSNewDirInpu
847787
pterm.Error.Println("fs service not available")
848788
return nil
849789
}
850-
if b.browsers == nil {
851-
pterm.Error.Println("browsers service not available")
852-
return nil
853-
}
854790
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
855791
if err != nil {
856792
return util.CleanedUpSdkError{Err: err}
@@ -875,10 +811,6 @@ func (b BrowsersCmd) FSDeleteDirectory(ctx context.Context, in BrowsersFSDeleteD
875811
pterm.Error.Println("fs service not available")
876812
return nil
877813
}
878-
if b.browsers == nil {
879-
pterm.Error.Println("browsers service not available")
880-
return nil
881-
}
882814
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
883815
if err != nil {
884816
return util.CleanedUpSdkError{Err: err}
@@ -899,10 +831,6 @@ func (b BrowsersCmd) FSDeleteFile(ctx context.Context, in BrowsersFSDeleteFileIn
899831
pterm.Error.Println("fs service not available")
900832
return nil
901833
}
902-
if b.browsers == nil {
903-
pterm.Error.Println("browsers service not available")
904-
return nil
905-
}
906834
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
907835
if err != nil {
908836
return util.CleanedUpSdkError{Err: err}
@@ -923,10 +851,6 @@ func (b BrowsersCmd) FSDownloadDirZip(ctx context.Context, in BrowsersFSDownload
923851
pterm.Error.Println("fs service not available")
924852
return nil
925853
}
926-
if b.browsers == nil {
927-
pterm.Error.Println("browsers service not available")
928-
return nil
929-
}
930854
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
931855
if err != nil {
932856
return util.CleanedUpSdkError{Err: err}
@@ -964,10 +888,6 @@ func (b BrowsersCmd) FSFileInfo(ctx context.Context, in BrowsersFSFileInfoInput)
964888
pterm.Error.Println("fs service not available")
965889
return nil
966890
}
967-
if b.browsers == nil {
968-
pterm.Error.Println("browsers service not available")
969-
return nil
970-
}
971891
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
972892
if err != nil {
973893
return util.CleanedUpSdkError{Err: err}
@@ -990,10 +910,6 @@ func (b BrowsersCmd) FSListFiles(ctx context.Context, in BrowsersFSListFilesInpu
990910
pterm.Error.Println("fs service not available")
991911
return nil
992912
}
993-
if b.browsers == nil {
994-
pterm.Error.Println("browsers service not available")
995-
return nil
996-
}
997913
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
998914
if err != nil {
999915
return util.CleanedUpSdkError{Err: err}
@@ -1023,10 +939,6 @@ func (b BrowsersCmd) FSMove(ctx context.Context, in BrowsersFSMoveInput) error {
1023939
pterm.Error.Println("fs service not available")
1024940
return nil
1025941
}
1026-
if b.browsers == nil {
1027-
pterm.Error.Println("browsers service not available")
1028-
return nil
1029-
}
1030942
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
1031943
if err != nil {
1032944
return util.CleanedUpSdkError{Err: err}
@@ -1047,10 +959,6 @@ func (b BrowsersCmd) FSReadFile(ctx context.Context, in BrowsersFSReadFileInput)
1047959
pterm.Error.Println("fs service not available")
1048960
return nil
1049961
}
1050-
if b.browsers == nil {
1051-
pterm.Error.Println("browsers service not available")
1052-
return nil
1053-
}
1054962
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
1055963
if err != nil {
1056964
return util.CleanedUpSdkError{Err: err}
@@ -1087,10 +995,6 @@ func (b BrowsersCmd) FSSetPermissions(ctx context.Context, in BrowsersFSSetPerms
1087995
pterm.Error.Println("fs service not available")
1088996
return nil
1089997
}
1090-
if b.browsers == nil {
1091-
pterm.Error.Println("browsers service not available")
1092-
return nil
1093-
}
1094998
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
1095999
if err != nil {
10961000
return util.CleanedUpSdkError{Err: err}
@@ -1118,10 +1022,6 @@ func (b BrowsersCmd) FSUpload(ctx context.Context, in BrowsersFSUploadInput) err
11181022
pterm.Error.Println("fs service not available")
11191023
return nil
11201024
}
1121-
if b.browsers == nil {
1122-
pterm.Error.Println("browsers service not available")
1123-
return nil
1124-
}
11251025
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
11261026
if err != nil {
11271027
return util.CleanedUpSdkError{Err: err}
@@ -1184,10 +1084,6 @@ func (b BrowsersCmd) FSUploadZip(ctx context.Context, in BrowsersFSUploadZipInpu
11841084
pterm.Error.Println("fs service not available")
11851085
return nil
11861086
}
1187-
if b.browsers == nil {
1188-
pterm.Error.Println("browsers service not available")
1189-
return nil
1190-
}
11911087
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
11921088
if err != nil {
11931089
return util.CleanedUpSdkError{Err: err}
@@ -1214,14 +1110,9 @@ func (b BrowsersCmd) FSWriteFile(ctx context.Context, in BrowsersFSWriteFileInpu
12141110
pterm.Error.Println("fs service not available")
12151111
return nil
12161112
}
1217-
if b.browsers == nil {
1218-
pterm.Error.Println("browsers service not available")
1219-
return nil
1220-
}
12211113
br, err := b.resolveBrowserByIdentifier(ctx, in.Identifier)
12221114
if err != nil {
1223-
pterm.Error.Printf("Failed to write file: %v\n", err)
1224-
return nil
1115+
return util.CleanedUpSdkError{Err: err}
12251116
}
12261117
if br == nil {
12271118
pterm.Error.Printf("Browser '%s' not found\n", in.Identifier)
@@ -1716,7 +1607,7 @@ func (b BrowsersCmd) resolveBrowserByIdentifier(ctx context.Context, identifier
17161607
if err != nil {
17171608
return nil, err
17181609
}
1719-
if browsers == nil || len(*browsers) == 0 {
1610+
if browsers == nil {
17201611
return nil, nil
17211612
}
17221613
for _, br := range *browsers {

0 commit comments

Comments
 (0)