diff --git a/.DS_Store b/.DS_Store new file mode 100644 index 00000000..5008ddfc Binary files /dev/null and b/.DS_Store differ diff --git a/main.go b/main.go index 44c32b3f..0b7cbfa9 100644 --- a/main.go +++ b/main.go @@ -98,8 +98,16 @@ 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 + // создайте объект ParcelStore функцией NewParcelStore + store := NewParcelStore(db) service := NewParcelService(store) // регистрация посылки diff --git a/parcel.go b/parcel.go index db6c815d..29a8e5bc 100644 --- a/parcel.go +++ b/parcel.go @@ -14,34 +14,76 @@ func NewParcelStore(db *sql.DB) ParcelStore { func (s ParcelStore) Add(p Parcel) (int, error) { // реализуйте добавление строки в таблицу parcel, используйте данные из переменной p - + 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 0, nil + return int(id), err } 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 Parcel{}, 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 []Parcel{}, err + } + defer rows.Close() // заполните срез Parcel данными из таблицы var res []Parcel + for rows.Next() { + var cl Parcel + + err := rows.Scan(&cl.Number, &cl.Client, &cl.Status, &cl.Address, &cl.CreatedAt) + if err != nil { + return []Parcel{}, err + } + res = append(res, cl) + } + + err = rows.Err() + if err != nil { + return []Parcel{}, err + } return res, nil } func (s ParcelStore) SetStatus(number int, status string) error { // реализуйте обновление статуса в таблице parcel - + _, err := s.db.Exec("UPDATE parcel SET status = :status WHERE number = :number", + sql.Named("status", status), + sql.Named("number", number)) + if err != nil { + return err + } return nil } @@ -49,6 +91,14 @@ func (s ParcelStore) SetAddress(number int, address string) error { // реализуйте обновление адреса в таблице parcel // менять адрес можно только если значение статуса registered + _, 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", "registered")) + if err != nil { + return err + } + return nil } @@ -56,5 +106,11 @@ func (s ParcelStore) Delete(number int) error { // реализуйте удаление строки из таблицы parcel // удалять строку можно только если значение статуса registered + _, err := s.db.Exec("DELETE FROM parcel WHERE number = :number AND status = :status", sql.Named("number", number), + sql.Named("status", "registered")) + if err != nil { + return err + } + return nil } diff --git a/parcel_test.go b/parcel_test.go index d1b93827..5bec4a8e 100644 --- a/parcel_test.go +++ b/parcel_test.go @@ -6,6 +6,7 @@ import ( "testing" "time" + "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" ) @@ -31,63 +32,135 @@ 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() + + _, err = db.Exec("DELETE FROM parcel") // - очистка + require.NoError(t, err) // - очистка + store := NewParcelStore(db) parcel := getTestParcel() // add - // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + + + num, err := store.Add(parcel) + require.NoError(t, err) + assert.NotEmpty(t, num) // get // получите только что добавленную посылку, убедитесь в отсутствии ошибки // проверьте, что значения всех полей в полученном объекте совпадают со значениями полей в переменной parcel + parcel.Number = num + p, err := store.Get(num) + require.NoError(t, err) + assert.Equal(t, parcel, p) // delete // удалите добавленную посылку, убедитесь в отсутствии ошибки // проверьте, что посылку больше нельзя получить из БД + err = store.Delete(num) + require.NoError(t, err) + + _, err = store.Get(num) + assert.Equal(t, sql.ErrNoRows, err) + } // TestSetAddress проверяет обновление адреса func TestSetAddress(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + + defer db.Close() + + _, err = db.Exec("DELETE FROM parcel") // - очистка + require.NoError(t, err) // - очистка + + store := NewParcelStore(db) + parcel := getTestParcel() // add // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + num, err := store.Add(parcel) + require.NoError(t, err) + assert.NotEmpty(t, num) // set address // обновите адрес, убедитесь в отсутствии ошибки newAddress := "new test address" + err = store.SetAddress(num, newAddress) + require.NoError(t, err) // check // получите добавленную посылку и убедитесь, что адрес обновился + parcelNew, err := store.Get(num) + require.NoError(t, err) + + parcel.Address = newAddress + parcel.Number = num + assert.Equal(t, parcel, parcelNew) } // TestSetStatus проверяет обновление статуса func TestSetStatus(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + + defer db.Close() + + _, err = db.Exec("DELETE FROM parcel") // - очистка + require.NoError(t, err) // - очистка + + store := NewParcelStore(db) + parcel := getTestParcel() // add // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + num, err := store.Add(parcel) + require.NoError(t, err) + assert.NotEmpty(t, num) // set status // обновите статус, убедитесь в отсутствии ошибки + newStatus := "new test status" + err = store.SetStatus(num, newStatus) + require.NoError(t, err) // check // получите добавленную посылку и убедитесь, что статус обновился + parcelNewS, err := store.Get(num) + require.NoError(t, err) + + parcel.Status = newStatus + parcel.Number = num + assert.Equal(t, parcelNewS, parcel) + } // TestGetByClient проверяет получение посылок по идентификатору клиента func TestGetByClient(t *testing.T) { // prepare - db, err := // настройте подключение к БД + db, err := sql.Open("sqlite", "tracker.db") + require.NoError(t, err) + + defer db.Close() + + _, err = db.Exec("DELETE FROM parcel") // - очистка + require.NoError(t, err) // - очистка + + store := NewParcelStore(db) parcels := []Parcel{ getTestParcel(), getTestParcel(), getTestParcel(), } + parcelMap := map[int]Parcel{} // задаём всем посылкам один и тот же идентификатор клиента @@ -97,8 +170,11 @@ func TestGetByClient(t *testing.T) { parcels[2].Client = client // add + // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора for i := 0; i < len(parcels); i++ { - id, err := // добавьте новую посылку в БД, убедитесь в отсутствии ошибки и наличии идентификатора + id, err := store.Add(parcels[i]) + require.NoError(t, err) + assert.NotEmpty(t, id) // обновляем идентификатор добавленной у посылки parcels[i].Number = id @@ -108,14 +184,19 @@ func TestGetByClient(t *testing.T) { } // get by client - storedParcels, err := // получите список посылок по идентификатору клиента, сохранённого в переменной client + storedParcels, err := store.GetByClient(client) + require.NoError(t, err) + assert.Equal(t, len(storedParcels), len(parcelMap)) + // получите список посылок по идентификатору клиента, сохранённого в переменной client // убедитесь в отсутствии ошибки // убедитесь, что количество полученных посылок совпадает с количеством добавленных // check - for _, parcel := range storedParcels { + for i := 0; i < len(parcelMap); i ++ { + assert.Equal(t, parcelMap[storedParcels[i].Number], storedParcels[i]) // в parcelMap лежат добавленные посылки, ключ - идентификатор посылки, значение - сама посылка // убедитесь, что все посылки из storedParcels есть в parcelMap // убедитесь, что значения полей полученных посылок заполнены верно + assert.NotEmpty(t, parcelMap[storedParcels[i].Number]) } } diff --git a/tracker.db b/tracker.db index b6ba48a1..3599dd31 100644 Binary files a/tracker.db and b/tracker.db differ