Fix coordinate editor exceed maxDegrees issue

offtherailz · offtherailz · commit db35bee4d23e · 2019-04-05T11:19:23.000+02:00
diff --git a/web/client/components/misc/coordinateeditors/editors/AeronauticalCoordinateEditor.jsx b/web/client/components/misc/coordinateeditors/editors/AeronauticalCoordinateEditor.jsx
@@ -59,6 +59,10 @@ class AeronauticalCoordinateEditor extends React.Component {
             let direction = newValues.direction;
             seconds = this.roundToNextSexagesimalStep(seconds);
             minutes = this.roundToNextSexagesimalStep(minutes);
+            if (degrees === this.props.maxDegrees) {
+                minutes = 0;
+                seconds = 0;
+            }
             direction = degrees < 0
                 ? (direction === this.props.directions[0] ? this.props.directions[1] : this.props.directions[0])
                 : direction;
diff --git a/web/client/components/misc/coordinateeditors/editors/__tests__/AeronauticalCoordinateEditor-test.jsx b/web/client/components/misc/coordinateeditors/editors/__tests__/AeronauticalCoordinateEditor-test.jsx
@@ -46,4 +46,26 @@ describe('AeronauticalCoordinateEditor enhancer', () => {
         expect(spyonChange).toHaveBeenCalled();
         expect(parseFloat(spyonChange.calls[0].arguments[0])).toBe(20);
     });
+    it('Test AeronauticalCoordinateEditor onChange not exceed maxDegrees', () => {
+        const actions = {
+            onChange: () => { }
+        };
+        const spyonChange = expect.spyOn(actions, 'onChange');
+        ReactDOM.render(<AeronauticalCoordinateEditor
+            coordinate="lon"
+            value={180}
+            onChange={actions.onChange} />, document.getElementById("container"));
+        const container = document.getElementById('container');
+        const elements = container.querySelectorAll('input');
+        expect(elements.length).toBe(3);
+        expect(elements[0].value).toBe('180');
+        expect(elements[1].value).toBe('0');
+        expect(elements[2].value).toBe('0');
+        ReactTestUtils.Simulate.change(elements[1], { target: { value: "20" } });
+        expect(spyonChange).toHaveBeenCalled();
+        expect(parseFloat(spyonChange.calls[0].arguments[0])).toBe(180);
+        expect(elements[0].value).toBe('180');
+        expect(elements[1].value).toBe('0');
+        expect(elements[2].value).toBe('0');
+    });
 });
