Fix nextjs resource name (#2834)

* add failing test

* fix test

---------

Co-authored-by: Selim Belhaouane <selim.belhaouane@garda.com>

packages/datadog-instrumentations/src/next.js

@@ -26,11 +26,13 @@ function wrapHandleApiRequest (handleApiRequest) {
       return promise.then(handled => {
         if (!handled) return handled
 
-        const page = getPageFromPath(pathname, this.dynamicRoutes)
+        return this.hasPage(pathname).then(pageFound => {
+          const page = pageFound ? pathname : getPageFromPath(pathname, this.dynamicRoutes)
 
-        pageLoadChannel.publish({ page })
+          pageLoadChannel.publish({ page })
 
-        return handled
+          return handled
+        })
       })
     })
   }

packages/datadog-plugin-next/test/index.spec.js

@@ -110,6 +110,28 @@ describe('Plugin', function () {
               .catch(done)
           })
 
+          const pathTests = [
+            ['/api/hello', '/api/hello'],
+            ['/api/hello/world', '/api/hello/[name]'],
+            ['/api/hello/other', '/api/hello/other']
+          ]
+          pathTests.forEach(([url, expectedPath]) => {
+            it(`should infer the corrrect resource path (${expectedPath})`, done => {
+              agent
+                .use(traces => {
+                  const spans = traces[0]
+
+                  expect(spans[0]).to.have.property('resource', `GET ${expectedPath}`)
+                })
+                .then(done)
+                .catch(done)
+
+              axios
+                .get(`http://localhost:${port}${url}`)
+                .catch(done)
+            })
+          })
+
           it('should propagate context', done => {
             axios
               .get(`http://localhost:${port}/api/hello/world`)
@@ -187,6 +209,28 @@ describe('Plugin', function () {
               .catch(done)
           })
 
+          const pathTests = [
+            ['/hello', '/hello'],
+            ['/hello/world', '/hello/[name]'],
+            ['/hello/other', '/hello/other']
+          ]
+          pathTests.forEach(([url, expectedPath]) => {
+            it(`should infer the corrrect resource (${expectedPath})`, done => {
+              agent
+                .use(traces => {
+                  const spans = traces[0]
+
+                  expect(spans[0]).to.have.property('resource', `GET ${expectedPath}`)
+                })
+                .then(done)
+                .catch(done)
+
+              axios
+                .get(`http://localhost:${port}${url}`)
+                .catch(done)
+            })
+          })
+
           it('should handle pages not found', done => {
             agent
               .use(traces => {

packages/datadog-plugin-next/test/pages/api/hello/index.js

@@ -0,0 +1,9 @@
+// Next.js API route support: https://nextjs.org/docs/api-routes/introduction
+
+export default (req, res) => {
+  const tracer = require('../../../../../dd-trace')
+  const span = tracer.scope().active()
+  const name = span && span.context()._name
+
+  res.status(200).json({ name })
+}

packages/datadog-plugin-next/test/pages/api/hello/other.js

@@ -0,0 +1,9 @@
+// Next.js API route support: https://nextjs.org/docs/api-routes/introduction
+
+export default (req, res) => {
+  const tracer = require('../../../../../dd-trace')
+  const span = tracer.scope().active()
+  const name = span && span.context()._name
+
+  res.status(200).json({ name })
+}

packages/datadog-plugin-next/test/pages/hello/[name].js

@@ -1,7 +1,7 @@
 export default function Home () {
   return (
     <div>
-      Hello World!
+      Hello [name]!
     </div>
   )
 }

packages/datadog-plugin-next/test/pages/hello/index.js

@@ -0,0 +1,7 @@
+export default function Home () {
+  return (
+    <div>
+      Hello index!
+    </div>
+  )
+}

packages/datadog-plugin-next/test/pages/hello/other.js

@@ -0,0 +1,7 @@
+export default function Home () {
+  return (
+    <div>
+      Hello other!
+    </div>
+  )
+}