Merge pull request #175 from dls-controls/no-initial-query

No initial query

Author: willrogers

src/connection/coniql.ts

@@ -188,38 +188,6 @@ function coniqlToDType(
   );
 }
 
-const PV_QUERY = gql`
-  query query1($pvName: ID!) {
-    getChannel(id: $pvName) {
-      id
-      time {
-        datetime
-      }
-      value {
-        string
-        float
-        base64Array {
-          numberType
-          base64
-        }
-      }
-      status {
-        quality
-        message
-        mutable
-      }
-      display {
-        units
-        form
-        controlRange {
-          max
-          min
-        }
-      }
-    }
-  }
-`;
-
 const PV_SUBSCRIPTION = gql`
   subscription sub1($pvName: ID!) {
     subscribeChannel(id: $pvName) {
@@ -347,17 +315,6 @@ export class ConiqlPlugin implements Connection {
   }
 
   private _subscribe(pvName: string): Subscription {
-    // Make a query to get the initial values.
-    // https://github.com/apollographql/subscriptions-transport-ws/issues/170
-    this.client
-      .query({
-        query: PV_QUERY,
-        variables: { pvName: pvName }
-      })
-      .then(data => {
-        this._process(data, pvName, "getChannel");
-      });
-    // Subscribe to further updates.
     return this.client
       .subscribe({
         query: PV_SUBSCRIPTION,

src/connection/forwarder.ts

@@ -13,8 +13,6 @@ export class ConnectionForwarder implements Connection {
     this.connected = false;
   }
   private getConnection(pvName: string): Connection {
-    console.log("getConnection");
-    console.log(pvName);
     for (const [prefix, connection] of this.prefixConnections) {
       if (pvName.startsWith(prefix)) {
         if (connection !== undefined) {

src/connection/sim.test.ts

@@ -86,7 +86,6 @@ it("test illegal names", (): void => {
 
 it("test enum", (): void => {
   getValue("sim://enum", (value: DType): void => {
-    console.log(value);
     expect(
       ["one", "two", "three", "four"].indexOf(DType.coerceString(value))
     ).toBeGreaterThan(-1);

src/redux/connectionMiddleware.ts

@@ -46,7 +46,6 @@ export const connectionMiddleware = (connection: Connection) => (
 
   switch (action.type) {
     case SUBSCRIBE: {
-      console.log(action.payload);
       const { pvName, type } = action.payload;
       // Are we already subscribed?
       const effectivePvName = connection.subscribe(pvName, type);

src/ui/widgets/EmbeddedDisplay/bobParser.test.ts

@@ -85,7 +85,6 @@ describe("opi widget parser", (): void => {
   </display>`;
   it("handles a missing dimension", (): void => {
     const display = parseBob(noXString, "xxx");
-    console.log(display);
     // Is this correct?
     expect(display.x).toEqual(undefined);
   });

src/ui/widgets/Slideshow/slideshow.test.tsx

@@ -26,7 +26,6 @@ describe("<Slideshow />", (): void => {
   });
 
   test("it renders only one child component", () => {
-    // console.log(mounted.find("#child-element").debug());
     expect(mounted.find("#child-element")).toHaveLength(1);
   });
 

src/ui/widgets/Slideshow/slideshow.tsx

@@ -132,7 +132,6 @@ export const SlideshowComponent = (
           flexGrow: 0
         }}
         onClick={(): void => {
-          console.log("Next button");
           setTransition(slideRightTransition);
           setChildIndex(
             nextChildIndex(childIndex, props.children?.length ?? 0)