Unused local variables in TranslationTest file

To push this patch, we need to check if it's just a matter of initialized variables that aren't used or there is missing code that needs to be restored.
We need to check in which commit those variables were introduced and which commit removed the code using those variables, if such commit has been done.

I found the issue that removed the code that used to use the variables, but I'm not sure if we should re-add what was removed or just remove the unused variables.

The code bellow was removed in issue #2557113: Make t() return a TranslationWrapper object to remove reliance on a static, unpredictable safe list:

-    $this->translationManager->expects($this->once())
-      ->method('translate')
-      ->with($values['value'], $arguments, $options);

@paulocs Good catch!

Given that the current code is the following one, it's the variables that need to be removed, not the code to be added.

  public function testGet(array $values, $expected) {
    $container = new ContainerBuilder();
    $container->set('string_translation', $this->translationManager);
    \Drupal::setContainer($container);
    $arguments = isset($values['arguments']) ? $values['arguments'] : [];
    $options = isset($values['context']) ? [
      'context' => $values['context'],
    ] : [];
    $annotation = new Translation($values);
    $this->assertSame($expected, (string) $annotation->get());
  }

$values is already passed to the constructor of the Translation class. There is no need to extract $arguments and >$options from $values.

The patch is removing the unused variables, which is the correct thing to do.

Committed 703dbe1 and pushed to 9.1.x. Thanks!