diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml new file mode 100644 index 00000000..e0ea0a63 --- /dev/null +++ b/.github/workflows/tests.yml @@ -0,0 +1,18 @@ +name: Test Coverage + +on: + push: + branches: [ "**" ] + pull_request: + branches: [ "**" ] + +jobs: + test-coverage: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v4 + - uses: actions/setup-go@v5 + with: + go-version: '1.21' + - name: Run tests + run: go test -v -count=1 ./... diff --git a/main.go b/main.go index 44c32b3f..fff7ec30 100644 --- a/main.go +++ b/main.go @@ -98,8 +98,14 @@ func (s ParcelService) Delete(number int) error { func main() { // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + if err != nil { + fmt.Println(err) + return + } + defer db.Close() - store := // создайте объект ParcelStore функцией NewParcelStore + store := NewParcelStore(db) service := NewParcelService(store) // регистрация посылки diff --git a/parcel.go b/parcel.go index db6c815d..2d90f3dd 100644 --- a/parcel.go +++ b/parcel.go @@ -13,48 +13,90 @@ func NewParcelStore(db *sql.DB) ParcelStore { } func (s ParcelStore) Add(p Parcel) (int, error) { - // реализуйте добавление строки в таблицу parcel, используйте данные из переменной p - - // верните идентификатор последней добавленной записи - return 0, nil + res, err := s.db.Exec( + "INSERT INTO parcel (client, status, address, created_at) VALUES (:client, :status, :address, :created_at)", + sql.Named("client", p.Client), + sql.Named("status", p.Status), + sql.Named("address", p.Address), + sql.Named("created_at", p.CreatedAt), + ) + if err != nil { + return 0, err + } + + id, err := res.LastInsertId() + if err != nil { + return 0, err + } + + return int(id), nil } func (s ParcelStore) Get(number int) (Parcel, error) { - // реализуйте чтение строки по заданному number - // здесь из таблицы должна вернуться только одна строка + row := s.db.QueryRow( + "SELECT number, client, status, address, created_at FROM parcel WHERE number = :number", + sql.Named("number", number), + ) - // заполните объект Parcel данными из таблицы p := Parcel{} + err := row.Scan(&p.Number, &p.Client, &p.Status, &p.Address, &p.CreatedAt) + if err != nil { + return p, err + } return p, nil } func (s ParcelStore) GetByClient(client int) ([]Parcel, error) { - // реализуйте чтение строк из таблицы parcel по заданному client - // здесь из таблицы может вернуться несколько строк + rows, err := s.db.Query( + "SELECT number, client, status, address, created_at FROM parcel WHERE client = :client", + sql.Named("client", client), + ) + if err != nil { + return nil, err + } + defer rows.Close() - // заполните срез Parcel данными из таблицы var res []Parcel + for rows.Next() { + p := Parcel{} + if err := rows.Scan(&p.Number, &p.Client, &p.Status, &p.Address, &p.CreatedAt); err != nil { + return nil, err + } + res = append(res, p) + } + + if err := rows.Err(); err != nil { + return nil, err + } return res, nil } func (s ParcelStore) SetStatus(number int, status string) error { - // реализуйте обновление статуса в таблице parcel - - return nil + _, err := s.db.Exec( + "UPDATE parcel SET status = :status WHERE number = :number", + sql.Named("status", status), + sql.Named("number", number), + ) + return err } func (s ParcelStore) SetAddress(number int, address string) error { - // реализуйте обновление адреса в таблице parcel - // менять адрес можно только если значение статуса registered - - return nil + _, err := s.db.Exec( + "UPDATE parcel SET address = :address WHERE number = :number AND status = :status", + sql.Named("address", address), + sql.Named("number", number), + sql.Named("status", ParcelStatusRegistered), + ) + return err } func (s ParcelStore) Delete(number int) error { - // реализуйте удаление строки из таблицы parcel - // удалять строку можно только если значение статуса registered - - return nil + _, err := s.db.Exec( + "DELETE FROM parcel WHERE number = :number AND status = :status", + sql.Named("number", number), + sql.Named("status", ParcelStatusRegistered), + ) + return err } diff --git a/parcel_test.go b/parcel_test.go index d1b93827..31de6005 100644 --- a/parcel_test.go +++ b/parcel_test.go @@ -6,6 +6,8 @@ import ( "testing" "time" + _ "modernc.org/sqlite" + "github.com/stretchr/testify/require" ) @@ -31,57 +33,94 @@ func getTestParcel() Parcel { // TestAddGetDelete проверяет добавление, получение и удаление посылки func TestAddGetDelete(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + defer db.Close() + store := NewParcelStore(db) parcel := getTestParcel() // add - // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + id, err := store.Add(parcel) + require.NoError(t, err) + require.NotZero(t, id) // get - // получите только что добавленную посылку, убедитесь в отсутствии ошибки - // проверьте, что значения всех полей в полученном объекте совпадают со значениями полей в переменной parcel + stored, err := store.Get(id) + require.NoError(t, err) + require.Equal(t, parcel.Client, stored.Client) + require.Equal(t, parcel.Status, stored.Status) + require.Equal(t, parcel.Address, stored.Address) + require.Equal(t, parcel.CreatedAt, stored.CreatedAt) + require.Equal(t, id, stored.Number) // delete - // удалите добавленную посылку, убедитесь в отсутствии ошибки - // проверьте, что посылку больше нельзя получить из БД + err = store.Delete(id) + require.NoError(t, err) + + _, err = store.Get(id) + require.ErrorIs(t, err, sql.ErrNoRows) } // TestSetAddress проверяет обновление адреса func TestSetAddress(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + defer db.Close() + + store := NewParcelStore(db) + parcel := getTestParcel() // add - // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + id, err := store.Add(parcel) + require.NoError(t, err) + require.NotZero(t, id) // set address - // обновите адрес, убедитесь в отсутствии ошибки newAddress := "new test address" + err = store.SetAddress(id, newAddress) + require.NoError(t, err) // check - // получите добавленную посылку и убедитесь, что адрес обновился + stored, err := store.Get(id) + require.NoError(t, err) + require.Equal(t, newAddress, stored.Address) } // TestSetStatus проверяет обновление статуса func TestSetStatus(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + defer db.Close() + + store := NewParcelStore(db) + parcel := getTestParcel() // add - // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + id, err := store.Add(parcel) + require.NoError(t, err) + require.NotZero(t, id) // set status - // обновите статус, убедитесь в отсутствии ошибки + err = store.SetStatus(id, ParcelStatusSent) + require.NoError(t, err) // check - // получите добавленную посылку и убедитесь, что статус обновился + stored, err := store.Get(id) + require.NoError(t, err) + require.Equal(t, ParcelStatusSent, stored.Status) } // TestGetByClient проверяет получение посылок по идентификатору клиента func TestGetByClient(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + defer db.Close() + + store := NewParcelStore(db) parcels := []Parcel{ getTestParcel(), @@ -98,7 +137,9 @@ func TestGetByClient(t *testing.T) { // add for i := 0; i < len(parcels); i++ { - id, err := // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + id, err := store.Add(parcels[i]) + require.NoError(t, err) + require.NotZero(t, id) // обновляем идентификатор добавленной у посылки parcels[i].Number = id @@ -108,14 +149,17 @@ func TestGetByClient(t *testing.T) { } // get by client - storedParcels, err := // получите список посылок по идентификатору клиента, сохранённого в переменной client - // убедитесь в отсутствии ошибки - // убедитесь, что количество полученных посылок совпадает с количеством добавленных + storedParcels, err := store.GetByClient(client) + require.NoError(t, err) + require.Len(t, storedParcels, len(parcels)) // check for _, parcel := range storedParcels { - // в parcelMap лежат добавленные посылки, ключ - идентификатор посылки, значение - сама посылка - // убедитесь, что все посылки из storedParcels есть в parcelMap - // убедитесь, что значения полей полученных посылок заполнены верно + expected, ok := parcelMap[parcel.Number] + require.True(t, ok) + require.Equal(t, expected.Client, parcel.Client) + require.Equal(t, expected.Status, parcel.Status) + require.Equal(t, expected.Address, parcel.Address) + require.Equal(t, expected.CreatedAt, parcel.CreatedAt) } }