Change country_code to null when setting lat/long to 0/null

core/streams/create.int.test.js

@@ -297,23 +297,58 @@ describe('POST /streams', () => {
     expect(stream.countryCode).toBe('GB')
   })
 
-  test('country code is null for undefined lat', async () => {
+  test('country code is null and timezone is UTC for undefined lat', async () => {
     const response = await request(app).post('/').send({ id: 'qwertyuiop40', name: 'my stream 4', latitude: undefined, longitude: -4.5 })
 
     expect(response.statusCode).toBe(201)
     const id = response.header.location.replace('/streams/', '')
     const stream = await models.Stream.findByPk(id)
     expect(stream.id).toBe('qwertyuiop40')
+    expect(stream.timezone).toBe('UTC')
     expect(stream.countryCode).toBe(null)
   })
 
-  test('country code is null for null lat', async () => {
+  test('country code is null and timezone is UTC for null lat', async () => {
     const response = await request(app).post('/').send({ id: 'qwertyuiop40', name: 'my stream 4', latitude: null, longitude: -4.5 })
 
     expect(response.statusCode).toBe(201)
     const id = response.header.location.replace('/streams/', '')
     const stream = await models.Stream.findByPk(id)
     expect(stream.id).toBe('qwertyuiop40')
+    expect(stream.timezone).toBe('UTC')
+    expect(stream.countryCode).toBe(null)
+  })
+
+  test('country code is null and timezone is UTC for null lat long', async () => {
+    const response = await request(app).post('/').send({ id: 'qwertyuiop40', name: 'my stream 4', latitude: null, longitude: null })
+
+    expect(response.statusCode).toBe(201)
+    const id = response.header.location.replace('/streams/', '')
+    const stream = await models.Stream.findByPk(id)
+    expect(stream.id).toBe('qwertyuiop40')
+    expect(stream.timezone).toBe('UTC')
+    expect(stream.countryCode).toBe(null)
+  })
+
+  test('country code is null and timezone is UTC for 0 lat long', async () => {
+    const response = await request(app).post('/').send({ id: 'qwertyuiop40', name: 'my stream 4', latitude: 0, longitude: 0 })
+
+    expect(response.statusCode).toBe(201)
+    const id = response.header.location.replace('/streams/', '')
+    const stream = await models.Stream.findByPk(id)
+    expect(stream.id).toBe('qwertyuiop40')
+    expect(stream.timezone).toBe('UTC')
+    expect(stream.countryCode).toBe(null)
+  })
+
+  test('country code is null and timezone is UTC for 0 lat', async () => {
+    const response = await request(app).post('/').send({ id: 'qwertyuiop40', name: 'my stream 4', latitude: 0, longitude: -4.5 })
+
+    expect(response.statusCode).toBe(201)
+    const id = response.header.location.replace('/streams/', '')
+    const stream = await models.Stream.findByPk(id)
+    expect(stream.id).toBe('qwertyuiop40')
+    expect(stream.timezone).toBe('UTC')
     expect(stream.countryCode).toBe(null)
   })
 

core/streams/dao/index.js

@@ -28,6 +28,7 @@ function computedAdditions (data, stream = {}) {
     additions.countryCode = getCountryCodeByLatLng(latitude, longitude)
   } else {
     additions.timezone = 'UTC'
+    additions.countryCode = null
   }
   return additions
 }

core/streams/dao/index.unit.test.js

@@ -4,7 +4,8 @@ describe('test computedAdditions function', () => {
   test('return an empty object if lat or lng is undefined', async () => {
     const additions = computedAdditions({ latitude: 54.2, longitude: undefined })
     expect(additions).toStrictEqual({
-      timezone: 'UTC'
+      timezone: 'UTC',
+      countryCode: null
     })
   })
   test('return expected result for the correct coordinates', async () => {

core/streams/update.int.test.js

@@ -236,7 +236,7 @@ describe('PATCH /streams/:id', () => {
     expect(streamUpdated.countryCode).toBe('PL')
   })
 
-  test('country code is not changed for undefined lat', async () => {
+  test('country code is changed to null and timezone is UTC for undefined lat', async () => {
     const stream = { id: 'qwertyuiop40', name: 'my stream 4', latitude: 54.2, longitude: -4.5, timezone: 'Europe/Britain', countryCode: 'GB', createdById: seedValues.primaryUserId }
     await models.Stream.create(stream)
     await models.UserStreamRole.create({ stream_id: stream.id, user_id: stream.createdById, role_id: seedValues.roleOwner })
@@ -245,10 +245,11 @@ describe('PATCH /streams/:id', () => {
     const response = await request(app).patch(`/${stream.id}`).send(requestBody)
     expect(response.statusCode).toBe(204)
     const streamUpdated = await models.Stream.findByPk(stream.id)
-    expect(streamUpdated.countryCode).toBe('GB')
+    expect(streamUpdated.timezone).toBe('UTC')
+    expect(streamUpdated.countryCode).toBeNull()
   })
 
-  test('country code is not changed for null lat', async () => {
+  test('country code is changed to null and timezone is UTC for null lat', async () => {
     const stream = { id: 'qwertyuiop40', name: 'my stream 4', latitude: 54.2, longitude: -4.5, timezone: 'Europe/Britain', countryCode: 'GB', createdById: seedValues.primaryUserId }
     await models.Stream.create(stream)
     await models.UserStreamRole.create({ stream_id: stream.id, user_id: stream.createdById, role_id: seedValues.roleOwner })
@@ -257,7 +258,47 @@ describe('PATCH /streams/:id', () => {
     const response2 = await request(app).patch(`/${stream.id}`).send(requestBody)
     expect(response2.statusCode).toBe(204)
     const streamUpdated = await models.Stream.findByPk(stream.id)
-    expect(streamUpdated.countryCode).toBe('GB')
+    expect(streamUpdated.timezone).toBe('UTC')
+    expect(streamUpdated.countryCode).toBeNull()
+  })
+
+  test('country code is changed to null and timezone is UTC for null lat long', async () => {
+    const stream = { id: 'qwertyuiop40', name: 'my stream 4', latitude: 54.2, longitude: -4.5, timezone: 'Europe/Britain', countryCode: 'GB', createdById: seedValues.primaryUserId }
+    await models.Stream.create(stream)
+    await models.UserStreamRole.create({ stream_id: stream.id, user_id: stream.createdById, role_id: seedValues.roleOwner })
+
+    const requestBody = { latitude: null, longitude: null }
+    const response2 = await request(app).patch(`/${stream.id}`).send(requestBody)
+    expect(response2.statusCode).toBe(204)
+    const streamUpdated = await models.Stream.findByPk(stream.id)
+    expect(streamUpdated.timezone).toBe('UTC')
+    expect(streamUpdated.countryCode).toBeNull()
+  })
+
+  test('country code is changed to null and timezone is UTC for 0 lat long', async () => {
+    const stream = { id: 'qwertyuiop40', name: 'my stream 4', latitude: 54.2, longitude: -4.5, timezone: 'Europe/Britain', countryCode: 'GB', createdById: seedValues.primaryUserId }
+    await models.Stream.create(stream)
+    await models.UserStreamRole.create({ stream_id: stream.id, user_id: stream.createdById, role_id: seedValues.roleOwner })
+
+    const requestBody = { latitude: 0, longitude: 0 }
+    const response2 = await request(app).patch(`/${stream.id}`).send(requestBody)
+    expect(response2.statusCode).toBe(204)
+    const streamUpdated = await models.Stream.findByPk(stream.id)
+    expect(streamUpdated.timezone).toBe('UTC')
+    expect(streamUpdated.countryCode).toBeNull()
+  })
+
+  test('country code is changed to null and timezone is UTC for 0 lat', async () => {
+    const stream = { id: 'qwertyuiop40', name: 'my stream 4', latitude: 54.2, longitude: -4.5, timezone: 'Europe/Britain', countryCode: 'GB', createdById: seedValues.primaryUserId }
+    await models.Stream.create(stream)
+    await models.UserStreamRole.create({ stream_id: stream.id, user_id: stream.createdById, role_id: seedValues.roleOwner })
+
+    const requestBody = { latitude: 0, longitude: -4.5 }
+    const response2 = await request(app).patch(`/${stream.id}`).send(requestBody)
+    expect(response2.statusCode).toBe(204)
+    const streamUpdated = await models.Stream.findByPk(stream.id)
+    expect(streamUpdated.timezone).toBe('UTC')
+    expect(streamUpdated.countryCode).toBeNull()
   })
 
   test('country code is null for coordinates somewhere in the ocean', async () => {