Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • T Trilinos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 936
    • Issues 936
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 22
    • Merge requests 22
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • James Willenbring
  • Trilinos
  • Issues
  • #695
Closed
Open
Issue created Oct 09, 2016 by James Willenbring@jmwilleOwner

Tpetra::Map: Generalize tests that depend unnecessarily on particular GO types

Created by: mhoemmen

@trilinos/tpetra

Find and generalize all Tpetra::Map tests that depend unnecessarily on particular GlobalOrdinal (GO) types. These tests live in tpetra/core/test/Map.

One example of such a test is Map_removeEmptyProcesses.cpp. There is no reason why this test needs LO=int and GO=long. Its choice of GO=long is particularly bad, since GO=long is not enabled by default (default GO types are currently int and long long, but see #74 (closed)). This means that most developers haven't even been building the test, let alone running it.

See my recent commit cd63103c for an example of what to do. That commit fixes the test in other ways, but the main changes with respect to this task are as follows:

  1. Make sure everything in the test lives in an anonymous namespace (always a good idea, to avoid namespace collisions)
  2. Use TEUCHOS_UNIT_TEST_TEMPLATE_3_DECL instead of TEUCHOS_UNIT_TEST, to make the test templated on LO, GO, NT (the Node type; name doesn't matter)
  3. Add test instantiations at the end, using the pattern in Map_replaceCommWithSubset.cpp (as of the above commit to the develop branch)
Assignee
Assign to
Time tracking