Skip to content

Commit 6793616

Browse files
committed
more specific env var name for openWeatherMaps
1 parent c281552 commit 6793616

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ npm install --omit=dev
2222

2323
npm start
2424

25-
*You will need your own API key for openwheatermaps to get weatherdata. Add it in an .env file at the root as WEATHER_API_KEY.*
25+
*You will need your own API key for openwheatermaps to get weatherdata. Add it in an .env file at the root as OWM_API_KEY.*
2626

2727
**Start this app up for development:**
2828

src/middleware/validation.middleware.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export const validateResult = (req, res, next) => {
99
}
1010

1111
export const hasOwmKey = (req, res, next) => {
12-
if (!process.env.WEATHER_API_KEY)
12+
if (!process.env.OWM_API_KEY)
1313
return res
1414
.status(500)
1515
.send({ message: "API key missing from environment variables" });

src/services/openWeatherMaps.service.mjs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export const currentWeather = async (query) =>
77
url: OWM_API_URL + '/weather',
88
params: {
99
...query,
10-
appid: process.env.WEATHER_API_KEY
10+
appid: process.env.OWM_API_KEY
1111
}
1212
});
1313

@@ -17,7 +17,7 @@ export const forecastWeather = async (query) =>
1717
url: OWM_API_URL + '/forecast',
1818
params: {
1919
...query,
20-
appid: process.env.WEATHER_API_KEY
20+
appid: process.env.OWM_API_KEY
2121
}
2222
});
2323

@@ -27,7 +27,7 @@ export const currentPollution = async (query) =>
2727
url: OWM_API_URL + '/air_pollution',
2828
params: {
2929
...query,
30-
appid: process.env.WEATHER_API_KEY
30+
appid: process.env.OWM_API_KEY
3131
}
3232
});
3333

@@ -37,6 +37,6 @@ export const forecastPollution = async (query) =>
3737
url: OWM_API_URL + '/air_pollution/forecast',
3838
params: {
3939
...query,
40-
appid: process.env.WEATHER_API_KEY
40+
appid: process.env.OWM_API_KEY
4141
}
4242
});

src/tests/api.test.mjs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ describe("API Routes", () => {
2323
lon: "-74.0060",
2424
};
2525
const originalEnv = process.env.ENVIRONMENT;
26-
const originalApiKey = process.env.WEATHER_API_KEY;
26+
const originalApiKey = process.env.OWM_API_KEY;
2727

2828
beforeAll(() => {
29-
process.env.WEATHER_API_KEY = 'test-key';
29+
process.env.OWM_API_KEY = 'test-key';
3030
process.env.ENVIRONMENT = 'test';
3131
});
3232

3333
afterAll((done) => {
34-
process.env.WEATHER_API_KEY = originalEnv;
34+
process.env.OWM_API_KEY = originalEnv;
3535
process.env.ENVIRONMENT = originalApiKey;
3636
server.close(done);
3737
});

0 commit comments

Comments
 (0)