Merge pull request #299 from Asartea/asartea-patch-1
07_tempConversion: Update tempConversion function's naming
This commit is contained in:
commit
8746ce056a
3 changed files with 16 additions and 18 deletions
|
@ -2,12 +2,12 @@
|
|||
|
||||
Write two functions that convert temperatures from Fahrenheit to Celsius, and vice versa:
|
||||
```
|
||||
ftoc(32) // fahrenheit to celsius, should return 0
|
||||
convertToCelsius(32) // fahrenheit to celsius, should return 0
|
||||
|
||||
ctof(0) // celsius to fahrenheit, should return 32
|
||||
convertToFahrenheit(0) // celsius to fahrenheit, should return 32
|
||||
```
|
||||
|
||||
Because we are human, we want the result temperature to be rounded to one decimal place: i.e., `ftoc(100)` should return `37.8` and not `37.77777777777778`.
|
||||
Because we are human, we want the result temperature to be rounded to one decimal place: i.e., `convertToCelsius(100)` should return `37.8` and not `37.77777777777778`.
|
||||
|
||||
This exercise asks you to create more than one function so the `module.exports` section of the spec file looks a little different this time. Nothing to worry about, we're just packaging both functions into a single object to be exported.
|
||||
|
||||
|
|
|
@ -1,13 +1,11 @@
|
|||
const ftoc = function() {
|
||||
|
||||
const convertToCelsius = function() {
|
||||
};
|
||||
|
||||
const ctof = function() {
|
||||
|
||||
const convertToFahrenheit = function() {
|
||||
};
|
||||
|
||||
// Do not edit below this line
|
||||
module.exports = {
|
||||
ftoc,
|
||||
ctof
|
||||
convertToCelsius,
|
||||
convertToFahrenheit
|
||||
};
|
||||
|
|
|
@ -1,25 +1,25 @@
|
|||
const {ftoc, ctof} = require('./tempConversion')
|
||||
const {convertToCelsius, convertToFahrenheit} = require('./tempConversion')
|
||||
|
||||
describe('ftoc', () => {
|
||||
describe('convertToCelsius', () => {
|
||||
test('works', () => {
|
||||
expect(ftoc(32)).toEqual(0);
|
||||
expect(convertToCelsius(32)).toEqual(0);
|
||||
});
|
||||
test.skip('rounds to 1 decimal', () => {
|
||||
expect(ftoc(100)).toEqual(37.8);
|
||||
expect(convertToCelsius(100)).toEqual(37.8);
|
||||
});
|
||||
test.skip('works with negatives', () => {
|
||||
expect(ftoc(-100)).toEqual(-73.3);
|
||||
expect(convertToCelsius(-100)).toEqual(-73.3);
|
||||
});
|
||||
});
|
||||
|
||||
describe('ctof', () => {
|
||||
describe('convertToFahrenheit', () => {
|
||||
test.skip('works', () => {
|
||||
expect(ctof(0)).toEqual(32);
|
||||
expect(convertToFahrenheit(0)).toEqual(32);
|
||||
});
|
||||
test.skip('rounds to 1 decimal', () => {
|
||||
expect(ctof(73.2)).toEqual(163.8);
|
||||
expect(convertToFahrenheit(73.2)).toEqual(163.8);
|
||||
});
|
||||
test.skip('works with negatives', () => {
|
||||
expect(ctof(-10)).toEqual(14);
|
||||
expect(convertToFahrenheit(-10)).toEqual(14);
|
||||
});
|
||||
});
|
||||
|
|
Loading…
Reference in a new issue