Rename method to in Monitor class and add tests for loading monitor data and handling errors (#44)

Author: KqLLL

lib/monitor.js

@@ -66,7 +66,7 @@ class Monitor {
         this._api = this.constructor._api;
     }
 
-    async data() {
+    async loadData() {
         return this._api.axios
             .get(this._api.monitorUrl(this.key))
             .then((res) => {

test/test.js

@@ -201,6 +201,31 @@ describe('Monitor', () => {
         expect(stub).to.be.calledWith('https://eu.cronitor.link/ping/apiKey123/a-key');
         done();
     });
+
+    it('should load monitor data', async function() {
+        const monitor = new cronitor.Monitor('a-key');
+        const mockResponse = { data: { name: 'Test Monitor', type: 'job' } };
+        const stub = sinon.stub(monitor._api.axios, 'get').resolves(mockResponse);
+        
+        const result = await monitor.loadData();
+        
+        expect(stub).to.be.calledWith(monitor._api.monitorUrl(monitor.key));
+        expect(result).to.deep.equal(mockResponse.data);
+        expect(monitor.data).to.deep.equal(mockResponse.data);
+    });
+
+    it('should return error response when loadData fails', async function() {
+        const monitor = new cronitor.Monitor('a-key');
+        const errorResponse = { status: 404, data: { error: 'Not Found' } };
+        const stub = sinon.stub(monitor._api.axios, 'get')
+            .rejects({ response: errorResponse });
+        
+        const result = await monitor.loadData();
+        
+        expect(stub).to.be.calledWith(monitor._api.monitorUrl(monitor.key));
+        expect(result).to.deep.equal(errorResponse);
+        expect(monitor.data).to.be.null;
+    });
 });
 
 describe('Event', () => {