apache/gravitino

[Improvement] Creating a namespace from just white space should not be allowed

Open

#7958 aperta il 7 ago 2025

Vedi su GitHub
 (8 commenti) (0 reazioni) (1 assegnatario)Java (887 fork)auto 404
good first issueimprovement

Metriche repository

Star
 (3058 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What would you like to be improved?

Update this test to the following, and the test will fail.

  @Test
  public void testFromStringInvalidArgs() {
    Assertions.assertThrows(IllegalArgumentException.class, () -> Namespace.fromString(null));
    Assertions.assertThrows(IllegalArgumentException.class, () -> Namespace.fromString(".a"));
    Assertions.assertThrows(IllegalArgumentException.class, () -> Namespace.fromString("a."));
    Assertions.assertThrows(IllegalArgumentException.class, () -> Namespace.fromString("a..b"));
    Assertions.assertThrows(IllegalArgumentException.class, () -> Namespace.fromString(" "));
  }

How should we improve?

Update Namespace.fromString to throw a IllegalArgumentException if given a namespace that is just whitespace

Guida contributor

Direzione di ricerca
Aggiungi un controllo in Namespace.fromString per rifiutare stringhe di soli spazi bianchi e aggiorna il relativo metodo di test testFromStringInvalidArgs per includere la nuova asserzione.
Tech stack
java
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Java
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80