chore: purge firebase implementation (#131)

go.mod

@@ -5,7 +5,6 @@ go 1.16
 require (
 	cloud.google.com/go/firestore v1.6.1
 	cloud.google.com/go/pubsub v1.21.0
-	firebase.google.com/go v3.13.0+incompatible
 	github.com/99designs/gqlgen v0.14.0
 	github.com/ahmetb/go-linq/v3 v3.2.0
 	github.com/asaskevich/govalidator v0.0.0-20210307081110-f21760c49a8d

pkg/clinical/application/common/helpers/helpers.go

@@ -11,10 +11,8 @@ import (
 	"github.com/savannahghi/clinical/pkg/clinical/application/common"
 	"github.com/savannahghi/clinical/pkg/clinical/domain"
 
-	fb "github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/firebase"
 	"github.com/savannahghi/converterandformatter"
 	"github.com/savannahghi/enumutils"
-	"github.com/savannahghi/firebasetools"
 	"github.com/savannahghi/scalarutils"
 	log "github.com/sirupsen/logrus"
 )
@@ -410,18 +408,6 @@ func ValidateEmail(
 	if !govalidator.IsEmail(email) {
 		return fmt.Errorf("invalid email format")
 	}
-
-	if optIn {
-		data := domain.EmailOptIn{
-			Email:   email,
-			OptedIn: optIn,
-		}
-		_, err := firebasetools.SaveDataToFirestore(
-			firestoreClient, fb.EmailOptInCollectionName, data)
-		if err != nil {
-			return fmt.Errorf("unable to save email opt in: %v", err)
-		}
-	}
 	return nil
 }
 

pkg/clinical/application/common/testutils/testhelpers.go

@@ -3,12 +3,10 @@ package testutils
 import (
 	"context"
 
-	"cloud.google.com/go/firestore"
 	"github.com/savannahghi/clinical/pkg/clinical/application/extensions"
 	"github.com/savannahghi/clinical/pkg/clinical/infrastructure"
 	"github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/fhir"
 	dataset "github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/fhirdataset"
-	fb "github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/firebase"
 	"github.com/savannahghi/clinical/pkg/clinical/infrastructure/services/openconceptlab"
 	"github.com/savannahghi/clinical/pkg/clinical/usecases"
 	"github.com/savannahghi/firebasetools"
@@ -21,12 +19,9 @@ func InitializeTestService(ctx context.Context) (usecases.Interactor, error) {
 	baseExtension := extensions.NewBaseExtensionImpl(fc)
 	repo := dataset.NewFHIRRepository()
 	fhir := fhir.NewFHIRStoreImpl(repo)
-	firestoreExt := fb.NewFirestoreClientExtension(&firestore.Client{})
-	fbExt := fb.NewFBClientExtensionImpl()
-	f := fb.NewFirebaseRepository(firestoreExt, fbExt)
 	ocl := openconceptlab.NewServiceOCL()
 
-	infrastructure := infrastructure.NewInfrastructureInteractor(baseExtension, repo, fhir, f, ocl)
+	infrastructure := infrastructure.NewInfrastructureInteractor(baseExtension, repo, fhir, ocl)
 
 	usecases := usecases.NewUsecasesInteractor(infrastructure)
 

pkg/clinical/infrastructure/datastore/fhirdataset/dataset_test.go

@@ -1,19 +1,12 @@
 package fhirdataset_test
 
 import (
-	"context"
-	"fmt"
 	"log"
 	"os"
 	"strings"
 	"testing"
-	"time"
 
-	"cloud.google.com/go/firestore"
-	"firebase.google.com/go/auth"
 	dataset "github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/fhirdataset"
-	fb "github.com/savannahghi/clinical/pkg/clinical/infrastructure/datastore/firebase"
-	"github.com/savannahghi/firebasetools"
 	"github.com/tj/assert"
 )
 
@@ -29,41 +22,11 @@ func TestMain(m *testing.M) {
 	os.Setenv("CLOUD_HEALTH_DATASET_ID", "datasetid")
 	os.Setenv("CLOUD_HEALTH_FHIRSTORE_ID", "fhirid")
 
-	// !NOTE!
-	// Under no circumstances should you remove this env var when testing
-	// You risk purging important collections, like our prod collections
-	os.Setenv("ROOT_COLLECTION_SUFFIX", fmt.Sprintf("clinical_ci_%v", time.Now().Unix()))
-	ctx := context.Background()
-	r := fb.Repository{} // They are nil
-	fsc, fbc := InitializeTestFirebaseClient(ctx)
-	if fsc == nil {
-		log.Printf("failed to initialize test FireStore client")
-		return
-	}
-	if fbc == nil {
-		log.Printf("failed to initialize test FireBase client")
-		return
-	}
-
-	purgeRecords := func() {
-		collections := []string{
-			r.GetEmailOptInCollectionName(),
-		}
-		for _, collection := range collections {
-			ref := fsc.Collection(collection)
-			_ = firebasetools.DeleteCollection(ctx, fsc, ref, 10)
-		}
-	}
-
-	// try clean up first
-	purgeRecords()
-
 	// do clean up
 	log.Printf("Running tests ...")
 	code := m.Run()
 
 	log.Printf("Tearing tests down ...")
-	purgeRecords()
 
 	// restore environment varibles to original values
 	os.Setenv(envOriginalValue, "ENVIRONMENT")
@@ -73,25 +36,6 @@ func TestMain(m *testing.M) {
 	os.Exit(code)
 }
 
-func InitializeTestFirebaseClient(ctx context.Context) (*firestore.Client, *auth.Client) {
-	fc := firebasetools.FirebaseClient{}
-	fa, err := fc.InitFirebase()
-	if err != nil {
-		log.Panicf("unable to initialize Firebase: %s", err)
-	}
-
-	fsc, err := fa.Firestore(ctx)
-	if err != nil {
-		log.Panicf("unable to initialize Firestore: %s", err)
-	}
-
-	fbc, err := fa.Auth(ctx)
-	if err != nil {
-		log.Panicf("can't initialize Firebase auth when setting up tests: %s", err)
-	}
-	return fsc, fbc
-}
-
 func TestRepository_CreateDataset(t *testing.T) {
 	fhirRepo := dataset.NewFHIRRepository()
 	tests := []struct {