remove refresh since it could cause additional billing. we want to define the costs before expose it.

Author: ali-hamud

dist/powerbi.js

@@ -106,14 +106,6 @@ export declare class Report extends embed.Embed implements IReportNode, IFiltera
      * Prints the active page of the report by invoking `window.print()` on the embed iframe component.
      */
     print(): Promise<void>;
-    /**
-     * Refreshes data sources for the report.
-     *
-     * ```javascript
-     * report.refresh();
-     * ```
-     */
-    refresh(): Promise<void>;
     /**
      * Removes all filters at the report level.
      *

dist/powerbi.js.map

@@ -174,23 +174,6 @@ export class Report extends embed.Embed implements IReportNode, IFilterable {
       });
   }
 
-  /**
-   * Refreshes data sources for the report.
-   * 
-   * ```javascript
-   * report.refresh();
-   * ```
-   */
-  refresh(): Promise<void> {
-    return this.service.hpm.post<models.IError[]>('/report/refresh', null, { uid: this.config.uniqueId }, this.iframe.contentWindow)
-      .then(response => {
-        return response.body;
-      })
-      .catch(response => {
-        throw response.body;
-      });
-  }
-
   /**
    * Removes all filters at the report level.
    * 

dist/powerbi.min.js

@@ -1154,26 +1154,6 @@ describe('Protocol', function () {
       });
     });
 
-    describe('refresh', function () {
-      it('POST /report/refresh returns 202 if the request is valid', function (done) {
-        // Arrange
-        iframeLoaded
-          .then(() => {
-            spyApp.refreshData.and.returnValue(Promise.resolve(null));
-            // Act
-            hpm.post<void>('/report/refresh', null)
-              .then(response => {
-                // Assert
-                expect(spyApp.refreshData).toHaveBeenCalled();
-                expect(response.statusCode).toEqual(202);
-                // Cleanup
-                spyApp.refreshData.calls.reset();
-                done();
-              });
-          });
-      });
-    });
-
     describe('filters (report level)', function () {
       it('GET /report/filters returns 200 with body as array of filters', function (done) {
         // Arrange
@@ -2214,36 +2194,6 @@ describe('SDK-to-HPM', function () {
       });
     });
 
-    describe('refresh', function () {
-      it('report.refresh() sends POST /report/refresh', function () {
-        // Arrange
-        spyHpm.post.and.returnValue(Promise.resolve({
-          body: {}
-        }));
-
-        // Act
-        report.refresh();
-
-        // Assert
-        expect(spyHpm.post).toHaveBeenCalledWith('/report/refresh', null, { uid: uniqueId }, iframe.contentWindow);
-      });
-
-      it('report.refresh() returns promise that resolves if the request is accepted', function (done) {
-        // Arrange
-        spyHpm.post.and.returnValue(Promise.resolve({
-          body: {}
-        }));
-
-        // Act
-        report.refresh()
-          .then(() => {
-            // Assert
-            expect(spyHpm.post).toHaveBeenCalledWith('/report/refresh', null, { uid: uniqueId }, iframe.contentWindow);
-            done();
-          });
-      });
-    });
-
     describe('reload', function () {
       it('report.reload() sends POST /report/load with configuration in body', function () {
         // Arrange
@@ -3125,24 +3075,6 @@ describe('SDK-to-MockApp', function () {
       });
     });
 
-    describe('refresh', function () {
-      it('report.refresh() returns promise that resolves with null if the report refresh command was accepted', function (done) {
-        // Arrange
-        iframeLoaded
-          .then(() => {
-            spyApp.refreshData.and.returnValue(Promise.resolve(null));
-            // Act
-            report.refresh()
-              .then(response => {
-                // Assert
-                expect(spyApp.refreshData).toHaveBeenCalled();
-                expect(response).toEqual(undefined);
-                done();
-              });
-          });
-      });
-    });
-
     describe('settings', function () {
       it('report.updateSettings(setting) returns promise that rejects with validation error if object is invalid', function (done) {
         // Arrange

dist/report.d.ts

@@ -19,7 +19,6 @@ export interface IApp {
   validateFilter(filter: models.IFilter): Promise<models.IError[]>;
   // Other
   print(): Promise<void>;
-  refreshData(): Promise<void>;
   exportData(): Promise<void>;
 }
 
@@ -42,7 +41,6 @@ export const mockAppSpyObj = {
   validateFilter: jasmine.createSpy("validateFilter").and.callFake(models.validateFilter),
   // Other
   print: jasmine.createSpy("print").and.returnValue(Promise.resolve(null)),
-  refreshData: jasmine.createSpy("refreshData").and.returnValue(Promise.resolve(null)),
   exportData: jasmine.createSpy("exportData").and.returnValue(Promise.resolve(null)),
 
   reset() {
@@ -59,7 +57,6 @@ export const mockAppSpyObj = {
     mockAppSpyObj.setFilters.calls.reset();
     mockAppSpyObj.validateFilter.calls.reset();
     mockAppSpyObj.print.calls.reset();
-    mockAppSpyObj.refreshData.calls.reset();
     mockAppSpyObj.exportData.calls.reset();
   }
 };

src/report.ts

@@ -206,11 +206,6 @@ export function setupEmbedMockApp(iframeContentWindow: Window, parentWindow: Win
       });
   });
 
-  router.post('/report/refresh', (req, res) => {
-    app.refreshData();
-    res.send(202);
-  });
-
   router.patch('/report/settings', (req, res) => {
     const uniqueId = req.headers['uid'];
     const settings = req.body;